WebGL (1 and 2) canvas data source with width/height >256 + Luminance => fail
Reported by
steph...@sketchfab.com,
Jun 15 2017
|
|||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.86 Safari/537.36 Steps to reproduce the problem: 1. Go to https://jsfiddle.net/spnc4vno/ 2. black result 3. What is the expected behavior? gradient What went wrong? black instead of gradient Did this work before? Yes no idea Does this work in other browsers? Yes Chrome version: 59.0.3071.86 Channel: stable OS Version: 10.0 Flash Version: If the width and height of the canvas is below or equals 256 it works fine, higher than that and I receive: [.Offscreen-For-WebGL-000002B1689CF5E0]GL ERROR :GL_INVALID_OPERATION : glCopySubTextureCHROMIUM: invalid dest internal format GL_LUMINANCE
,
Jun 16 2017
Able to reproduce on Windows-10, Ubuntu 14.04 and Mac OS 10.12.5 using chrome latest stable #59.0.3071.86. Bisect Information: -------------------- Good build: 59.0.3066.0 Bad Build : 59.0.3068.1 You are probably looking for a change made after 463268 (known good), but no later than 463269 (first known bad). Change Log URL: https://chromium.googlesource.com/chromium/src/+log/0338b35867900b81de5e6960eec0572f152974b7..03b5cd7ff9b0c8457efe2578d18de91141be30b6 From the above change log suspecting below change Review-Url: https://codereview.chromium.org/2738163002 qiankun.miao/kbr@ - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner. Since this is a recent regression broke in M-59 adding RB-Stable, Please feel free to remove if this is not the case. Thanks!
,
Jun 17 2017
It's almost certain that using the GPU-to-GPU copy path for this case regressed it because the 256-size boundary is the point at which 2D canvas contexts are accelerated. Just need to debug the code path and fix it.
,
Jun 19 2017
Sorry, but since this made it out to Stable it is by definition not a release blocker for Stable. Removing the label. Submitter, I strongly encourage you to test with Chrome Beta and Dev (or Canary) to catch issues like this earlier.
,
Jul 10 2017
,
Jul 10 2017
,
Jul 10 2017
,
Jul 13 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/b642ce1b14a2b5d9ef78fd1b9ae2b5d6ae1c9cb9 commit b642ce1b14a2b5d9ef78fd1b9ae2b5d6ae1c9cb9 Author: Kenneth Russell <kbr@chromium.org> Date: Thu Jul 13 05:33:44 2017 Support LUMINANCE, ALPHA, and LUMINANCE_ALPHA in CopyTextureCHROMIUM. Use the luminance/alpha emulation blitter for CopyTex{Sub}Image2D on the Core Profile to support these. Simplified the copy shader and fixed bugs in handling of premul/unmul which are tested by the new tests in https://github.com/KhronosGroup/WebGL/pull/2451 . BUG= 733599 Change-Id: Ic52b318d917dd95b8eb2c8fa3ce7b2aaebc24a63 Reviewed-on: https://chromium-review.googlesource.com/567306 Commit-Queue: Kenneth Russell <kbr@chromium.org> Reviewed-by: Zhenyao Mo <zmo@chromium.org> Cr-Commit-Position: refs/heads/master@{#486281} [modify] https://crrev.com/b642ce1b14a2b5d9ef78fd1b9ae2b5d6ae1c9cb9/gpu/command_buffer/service/gles2_cmd_apply_framebuffer_attachment_cmaa_intel.cc [modify] https://crrev.com/b642ce1b14a2b5d9ef78fd1b9ae2b5d6ae1c9cb9/gpu/command_buffer/service/gles2_cmd_copy_texture_chromium.cc [modify] https://crrev.com/b642ce1b14a2b5d9ef78fd1b9ae2b5d6ae1c9cb9/gpu/command_buffer/service/gles2_cmd_copy_texture_chromium.h [modify] https://crrev.com/b642ce1b14a2b5d9ef78fd1b9ae2b5d6ae1c9cb9/gpu/command_buffer/service/gles2_cmd_decoder.cc
,
Jul 13 2017
,
Jul 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d36daf872d9601f32503e3c1f101e8d1395a4aa5 commit d36daf872d9601f32503e3c1f101e8d1395a4aa5 Author: Kenneth Russell <kbr@chromium.org> Date: Fri Jul 14 21:13:41 2017 Roll WebGL 5e57726..72eda82 https://chromium.googlesource.com/external/khronosgroup/webgl.git/+log/5e57726..72eda82 BUG= 733599 , 735483, angleproject:2103 TEST=bots TBR=zmo@chromium.org CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I4542a243a3b34141becea64e1f024872f64c3ce2 Reviewed-on: https://chromium-review.googlesource.com/570761 Commit-Queue: Kenneth Russell <kbr@chromium.org> Reviewed-by: Kenneth Russell <kbr@chromium.org> Cr-Commit-Position: refs/heads/master@{#486889} [modify] https://crrev.com/d36daf872d9601f32503e3c1f101e8d1395a4aa5/DEPS [modify] https://crrev.com/d36daf872d9601f32503e3c1f101e8d1395a4aa5/content/test/gpu/gpu_tests/webgl2_conformance_expectations.py [modify] https://crrev.com/d36daf872d9601f32503e3c1f101e8d1395a4aa5/content/test/gpu/gpu_tests/webgl_conformance_expectations.py [modify] https://crrev.com/d36daf872d9601f32503e3c1f101e8d1395a4aa5/content/test/gpu/gpu_tests/webgl_conformance_revision.txt
,
Jul 14 2017
This will be fixed in Chrome 61.
,
Aug 8 2017
,
Aug 29
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by steph...@sketchfab.com
, Jun 15 2017