commit e076a2327f0d082d46683f32614c495e049a4bb2
author Geoff Lang <geofflang@chromium.org> Mon Jul 16 15:34:05 2018 -0400
committer Commit Bot <commit-bot@chromium.org> Mon Jul 16 20:15:15 2018 +0000
tree 88b541734e947557220a9fb9524771687381963b
parent 242c4fe86801978907bb7794b7b13f75eccc558a

Vulkan: Fix ReadPixels checking the wrong FBO for flipping.

Only the read framebuffer should affect the flipping for ReadPixels.

BUG=angleproject:2673

Change-Id: I3733bb823f5f3ebd55df63a36cc99b510dc0b1aa
Reviewed-on: https://chromium-review.googlesource.com/1138817
Reviewed-by: Jamie Madill <jmadill@chromium.org>
Commit-Queue: Geoff Lang <geofflang@chromium.org>

src/libANGLE/renderer/vulkan/FramebufferVk.cpp

diff --git a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
index c106787..35009c0 100644
--- a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
+++ b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
@@ -308,7 +308,7 @@
         return gl::NoError();
     }
     gl::Rectangle flippedArea = clippedArea;
-    if (contextVk->isViewportFlipEnabledForDrawFBO())
+    if (contextVk->isViewportFlipEnabledForReadFBO())
     {
         flippedArea.y = fbRect.height - flippedArea.y - flippedArea.height;
     }
@@ -319,7 +319,7 @@
     ANGLE_TRY(beginWriteResource(contextVk, &commandBuffer));
 
     gl::PixelPackState packState(glState.getPackState());
-    if (contextVk->isViewportFlipEnabledForDrawFBO())
+    if (contextVk->isViewportFlipEnabledForReadFBO())
     {
         packState.reverseRowOrder = !packState.reverseRowOrder;
     }