Issue metadata
Sign in to add a comment
|
Canvas Bug
Reported by
ryanosw...@gmail.com,
Sep 13
|
||||||||||||||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.92 Safari/537.36 Steps to reproduce the problem: Go to www.karmata.com (this is my pre-launch webapp) Resize window vertically until all text disappears. It would take a very long time to build a reduced test case, but I'm happy to provide support if needed. What is the expected behavior? texsubimage2d and teximage2d uploads of 2d canvases are successful What went wrong? texsubimage2d and teximage2d uploads of 2d canvases fail silently. Did this work before? Yes 574606 Does this work in other browsers? Yes Chrome version: 69.0.3497.92 Channel: stable OS Version: 10.0 Flash Version: works fine at 574606, broken at 574616 Of these 10 changes, the following is immediately suspect. https://chromium.googlesource.com/chromium/src/+/f8b8872f2080130c2b5bdf026f32f46aad7598da
,
Sep 13
RB-Stable for M-69 as this is regressed in M-69, feel free to remove if this should not be blocking.
,
Sep 13
Hmm, this looks tricky. Adding in Canvas folks, fserb@ is out - is someone else comfortable looking at this regression ASAP?
,
Sep 13
I'm 99% sure this is a duplicate of Issue 878545 . A fix is almost complete which is small and which could be merged back. The application requires a login and I can't reproduce the behavior from the login screen. Please send me an invitation link (at kbr at chromium.org) if we need to log in to the app to reproduce and confirm the fix.
,
Sep 13
,
Sep 13
The issue does not require login to reproduce. Just resize the window up and down until all the text disappears.
,
Sep 13
Strangely, I seem to be having issues reproducing the bug on desktop, but the issues persist on mobile where resize is not required to trigger the bug. On mobile, the issue seems to happen only on textures that have had a LOT of draw calls, which seems consistent with kbr@'s conclusion that this is a dupe of Issue 878545 . I will work with kbr@ to verify that his fix addresses this issue as well.
,
Sep 14
Reporter: please watch Issue 878545 and help us test the new Canary (71.0.3551.3 or later) tomorrow or whenever it comes out on Android, and ensure that your app is fixed with it. Thanks.
,
Sep 14
Do you have a sense for when the patch for 69.x would get released?
,
Sep 14
Once the fix is verified on Canary, it'll be backported to 69. Please help us test Canary tomorrow. Thanks.
,
Sep 14
Latest release works fine. Feel free to close this issue since it is a duplicate of 878545.
,
Sep 14
Thanks submitter for re-testing. The merge back to M69 was non-trivial, so to avoid the possibility of destabilizing Chrome we are not going to do it. The fix will be in Chrome 70. Submitter, in the future, please test with Chrome Beta at least (if not also Chrome Canary) and file bugs that you see earlier so we can have a chance to fix them sooner. Thanks.
,
Sep 14
,
Sep 14
In that case, can you rollback the original offending change on chrome 69 instead? While we are happy to see prompt responses by the chrome team, it is extremely disappointing that we may have to wait an entire month for a resolution. If the chrome team views canvas APIs as second class citizens, we may regret our decision to even attempt building a webgl-enabled product. It seems a bit unreasonable to expect startups to accept the burden of regression testing browser releases, but this may be the cost we have to pay as early adopters.
,
Sep 14
It is not possible to roll back the M69 change which caused this regression; many other changes were built on top of it. |
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by kkaluri@chromium.org
, Sep 13Labels: -Pri-2 hasbisect-per-revision M-69 Target-70 Target-71 RegressedIn-69 M-71 M-70 Target-69 FoundIn-69 OS-Linux OS-Mac Pri-1
Owner: junov@chromium.org
Status: Assigned (was: Unconfirmed)