Commit b221486a by Jamie Madill Committed by Commit Bot

Vulkan: Fix TexSubImage2D state change test.

Was simply a matter of triggering a new write node in the subImage call. Bug: angleproject:2495 Change-Id: I06334ba6ee816f671e5c599c8e8f079f56adb25e Reviewed-on: https://chromium-review.googlesource.com/1028729 Commit-Queue: Jamie Madill <jmadill@chromium.org> Reviewed-by: 's avatarLuc Ferron <lucferron@chromium.org>
parent 0086de17
...@@ -271,6 +271,10 @@ gl::Error TextureVk::setSubImage(const gl::Context *context, ...@@ -271,6 +271,10 @@ gl::Error TextureVk::setSubImage(const gl::Context *context,
ANGLE_TRY(mPixelBuffer.stageSubresourceUpdate( ANGLE_TRY(mPixelBuffer.stageSubresourceUpdate(
contextVk, index, gl::Extents(area.width, area.height, area.depth), contextVk, index, gl::Extents(area.width, area.height, area.depth),
gl::Offset(area.x, area.y, area.z), formatInfo, unpack, type, pixels)); gl::Offset(area.x, area.y, area.z), formatInfo, unpack, type, pixels));
// Create a new graph node to store image initialization commands.
getNewWritingNode(contextVk->getRenderer());
return gl::NoError(); return gl::NoError();
} }
......
...@@ -1369,10 +1369,6 @@ TEST_P(SimpleStateChangeTest, RebindTextureDrawAgain) ...@@ -1369,10 +1369,6 @@ TEST_P(SimpleStateChangeTest, RebindTextureDrawAgain)
// correctly. // correctly.
TEST_P(SimpleStateChangeTest, DrawWithTextureTexSubImageThenDrawAgain) TEST_P(SimpleStateChangeTest, DrawWithTextureTexSubImageThenDrawAgain)
{ {
// TODO(jmadill): Start a new command buffer when another draw is called after doing a texture
// update to support this use case. http://anglebug.com/2495
ANGLE_SKIP_TEST_IF(IsVulkan());
GLuint program = get2DTexturedQuadProgram(); GLuint program = get2DTexturedQuadProgram();
ASSERT_NE(0u, program); ASSERT_NE(0u, program);
glUseProgram(program); glUseProgram(program);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment