Chrome incorrectly allows depth texture cubemaps in WebGL1 with WEBGL_depth_texture |
||
Issue descriptionChrome Version: 60 OS: all What steps will reproduce the problem? (1) Run this sample https://jsfiddle.net/greggman/z9zz0drg/ What is the expected result? trying to create a cubemap depth texture fails with INVALID_OPERATION What happens instead? creating the depth texture cubemap does not fail From the spec https://www.khronos.org/registry/webgl/extensions/WEBGL_depth_texture/ > The error INVALID_OPERATION is generated in the following situations: > > * texImage2D is called with format and internalformat of DEPTH_COMPONENT > or DEPTH_STENCIL and target is not TEXTURE_2D,
,
May 5 2017
Here is why we don't have test coverage: https://github.com/KhronosGroup/WebGL/pull/2394
,
May 9 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d5523a250fa3faed2930cabcc13ac0746c1cfb2f commit d5523a250fa3faed2930cabcc13ac0746c1cfb2f Author: zmo <zmo@chromium.org> Date: Tue May 09 18:15:16 2017 Generate INVALID_OPERATION when depth/stencil textures are specified using targets other than TEXTURE_2D in ES2/WebGL1. Also, Roll in related WebGL conformance test to have test coverage: https://chromium.googlesource.com/external/khronosgroup/webgl.git/+/73b300f24942adf7013de30ccdc6a2cc88105e72 BUG=718846 TEST=webgl_conformance R=piman@chromium.org CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Review-Url: https://codereview.chromium.org/2865233002 Cr-Commit-Position: refs/heads/master@{#470386} [modify] https://crrev.com/d5523a250fa3faed2930cabcc13ac0746c1cfb2f/DEPS [modify] https://crrev.com/d5523a250fa3faed2930cabcc13ac0746c1cfb2f/content/test/gpu/gpu_tests/webgl_conformance_revision.txt [modify] https://crrev.com/d5523a250fa3faed2930cabcc13ac0746c1cfb2f/gpu/command_buffer/service/texture_manager.cc |
||
►
Sign in to add a comment |
||
Comment 1 by zmo@chromium.org
, May 5 2017Labels: -Pri-3 Pri-2