commit 7f2520f13d2e7714002536c1c97f600bdddca246
author Jamie Madill <jmadill@chromium.org> Wed Jun 26 11:18:33 2019 -0400
committer Commit Bot <commit-bot@chromium.org> Wed Jun 26 16:35:03 2019 +0000
tree 9f1995c3fe51212bed7a98eb50282f9cdbb912cb
parent 8929b9c551b43840efe4a108c0fb8944ed7f8575

Vulkan: Refactor DynamicBuffer::init.

This will allow us to more easily create a white box test that sets a
very small initial size for a dynamic buffer.

Bug: angleproject:3082
Change-Id: Ic02bbee83ee8e0f4bfe182e9448c2ce60dea66d5
Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/1667645
Commit-Queue: Jamie Madill <jmadill@chromium.org>
Reviewed-by: Tobin Ehlis <tobine@google.com>

src/libANGLE/renderer/vulkan/FramebufferVk.cpp

diff --git a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
index ee031a6..b0b6645 100644
--- a/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
+++ b/src/libANGLE/renderer/vulkan/FramebufferVk.cpp
@@ -162,14 +162,10 @@
 FramebufferVk::FramebufferVk(RendererVk *renderer,
                              const gl::FramebufferState &state,
                              WindowSurfaceVk *backbuffer)
-    : FramebufferImpl(state),
-      mBackbuffer(backbuffer),
-      mActiveColorComponents(0),
-      mReadPixelBuffer(VK_BUFFER_USAGE_TRANSFER_DST_BIT, kMinReadPixelsBufferSize, true),
-      mBlitPixelBuffer(VK_BUFFER_USAGE_TRANSFER_SRC_BIT, kMinReadPixelsBufferSize, true)
+    : FramebufferImpl(state), mBackbuffer(backbuffer), mActiveColorComponents(0)
 {
-    mBlitPixelBuffer.init(1, renderer);
-    mReadPixelBuffer.init(4, renderer);
+    mReadPixelBuffer.init(renderer, VK_BUFFER_USAGE_TRANSFER_DST_BIT, 4, kMinReadPixelsBufferSize,
+                          true);
 }
 
 FramebufferVk::~FramebufferVk() = default;
@@ -180,7 +176,6 @@
     mFramebuffer.release(contextVk);
 
     mReadPixelBuffer.release(contextVk);
-    mBlitPixelBuffer.release(contextVk);
 }
 
 angle::Result FramebufferVk::discard(const gl::Context *context,