Vulkan: Swap RenderArea coords for pre-rotation

Fixes Vulkan validation-layer errors. Found by dEQP tests that use small viewports and/or scissors (see Buganizer bugs for details). Bug: angleproject:4431 Bug: b/157933235 Bug: b/157933198 Change-Id: I83966906d70c30a4b50209cf1a33649bf69e3dd8 Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2225607 Commit-Queue: Ian Elliott <ianelliott@google.com> Reviewed-by: Cody Northrop <cnorthrop@google.com>

Vulkan: Swap RenderArea coords for pre-rotation
1df70f03 · Ian Elliott · Commit Bot · ab372311 · 1df70f03
Commit 1df70f03 authored Jun 01, 2020 by Ian Elliott Committed by Commit Bot Jun 02, 2020
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

ContextVk.cpp src/libANGLE/renderer/vulkan/ContextVk.cpp +2 -2

No files found.
--- a/src/libANGLE/renderer/vulkan/ContextVk.cpp
+++ b/src/libANGLE/renderer/vulkan/ContextVk.cpp
@@ -4160,8 +4160,8 @@ angle::Result ContextVk::flushAndBeginRenderPass(
    if (isRotatedAspectRatioForDrawFBO())
    {
        // The surface is rotated 90/270 degrees.  This changes the aspect ratio of
-        // the surface.  Swap the width and height of the renderArea.
-        // TODO(ianelliott): handle small viewport/scissor cases.  http://anglebug.com/4431
+        // the surface.  Swap the x and y axis of the renderArea.
+        std::swap(rotatedRenderArea.x, rotatedRenderArea.y);
        std::swap(rotatedRenderArea.width, rotatedRenderArea.height);
    }