Regression: Tab crash is observed on 'https://davidwalsh.name/webgl-demo'
Reported by
dmascare...@etouch.net,
Jan 10 2017
|
||||||||||
Issue descriptionChrome Version:57.0.2977.0 (Official Build) 20e4bfed4baf6f6eac0d7142bb9e763bc11512e4-refs/heads/master@{#442447} OS : Mac(10.12.1, 10.11.6, 10.12), Windows(7,8,10), Test url: https://davidwalsh.name/webgl-demo What steps will reproduce the problem? 1. Launch chrome and navigate to above test url,observe. Actual: Tab crash is observed. Expected: Tab should not crash. This is regression issue, broken in 'M 57' and below is manual bisect info: Good build:57.0.2976.5 Bad build:57.0.2977.0 Crash id: Crash ID a5f98a58-76fc-4a14-a202-81928db31e11 (Server ID: ae0b0bd080000000) Crash ID a86b0dec-7d79-4101-af94-7479a82716f5 (Server ID: 02fe0bd080000000)
,
Jan 10 2017
This is #1 renderer crash on the latest canary(57.0.2977.0 -15 crashes from 15 clients) of Mac based on the available crash data. Windows canary is still not pushed to the users, will update the crash server behavior once the crash data comes in. Marking this as Dev blocker based on available crash data and recent regression. Link to the list of the builds: ============================== https://goto.google.com/zfoel
,
Jan 10 2017
Note: Above issue is reproducible on Linux OS also but need to reload the webpage
,
Jan 10 2017
Able to reproduce the issue on Ubuntu 14.04. Thank You!
,
Jan 10 2017
Users experienced this crash on the following builds: Mac Canary 57.0.2977.0 - 237.79 CPM, 155 reports, 133 clients (signature blink::CompositedLayerMapping::pixelSnappedCompositedBounds) If this update was incorrect, please add "Fracas-Wrong" label to prevent future updates. - Go/Fracas
,
Jan 10 2017
I can reproduce on macOS 10.12.2, Canary 57.0.2977.0, Intel HD 6000. Interesting note: the page does NOT crash if the tab is in the background while loading. If I then switch to it, it seems to be fine. So it seems to be triggered only if visible during load.
,
Jan 10 2017
Re comment 6: did you reproduce by just loading the page?
,
Jan 10 2017
Yes, I didn't have to do anything special. It crashed after about 2 seconds of loading.
,
Jan 10 2017
,
Jan 11 2017
Hit this repeatedly on Windows and Mac 57.0.2977.0 at http://www.reuters.com/article/us-usa-congress-hearings-postponed-idUSKBN14U2U2
,
Jan 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e00ffecc157dfa92f60c583cd3bfab7f102b65ae commit e00ffecc157dfa92f60c583cd3bfab7f102b65ae Author: chrishtr <chrishtr@chromium.org> Date: Wed Jan 11 03:22:11 2017 Return the enclosing composited PaintLayer rather than containingBlock for floats. A previous patch failed to return the composited PaintLayer, which is what GraphicsLayerUpdater expected. BUG= 679667 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://codereview.chromium.org/2624843002 Cr-Commit-Position: refs/heads/master@{#442778} [modify] https://crrev.com/e00ffecc157dfa92f60c583cd3bfab7f102b65ae/third_party/WebKit/Source/core/layout/compositing/GraphicsLayerUpdater.cpp [modify] https://crrev.com/e00ffecc157dfa92f60c583cd3bfab7f102b65ae/third_party/WebKit/Source/core/paint/PaintLayer.cpp [modify] https://crrev.com/e00ffecc157dfa92f60c583cd3bfab7f102b65ae/third_party/WebKit/Source/core/paint/PaintLayerTest.cpp
,
Jan 11 2017
,
Jan 11 2017
CL from C#11 looks like missed today's canary(57.0.2978.0 - revision@442756). Will have to wait for next canary to verify this issue. Note: Crashes seen on the latest canary(57.0.2977.0) of Chrome OS and Android as well.
,
Jan 11 2017
chrishtr@, could you please merge the above fix to 2977 branch? I am planning to prepare a back-up build for this week's Dev release. PS: You do not need to follow any 'Merge-Request' process since 2977 is yet to be branched officially. Thank you!
,
Jan 11 2017
ok
,
Jan 11 2017
oops, sorry, I meant to be 2978 instead of 2977.
,
Jan 11 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e9d7b27fb83ad0bfcdc303d5228c5a51d68f526e commit e9d7b27fb83ad0bfcdc303d5228c5a51d68f526e Author: Chris Harrelson <chrishtr@chromium.org> Date: Wed Jan 11 18:24:48 2017 Return the enclosing composited PaintLayer rather than containingBlock for floats. A previous patch failed to return the composited PaintLayer, which is what GraphicsLayerUpdater expected. BUG= 679667 CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Review-Url: https://codereview.chromium.org/2624843002 Cr-Commit-Position: refs/heads/master@{#442778} (cherry picked from commit e00ffecc157dfa92f60c583cd3bfab7f102b65ae) Review-Url: https://codereview.chromium.org/2621343003 . Cr-Commit-Position: refs/branch-heads/2978@{#3} Cr-Branched-From: e2867aed46b3f00f3735d310144d6ef638f8be1a-refs/heads/master@{#442756} [modify] https://crrev.com/e9d7b27fb83ad0bfcdc303d5228c5a51d68f526e/third_party/WebKit/Source/core/layout/compositing/GraphicsLayerUpdater.cpp [modify] https://crrev.com/e9d7b27fb83ad0bfcdc303d5228c5a51d68f526e/third_party/WebKit/Source/core/paint/PaintLayer.cpp [modify] https://crrev.com/e9d7b27fb83ad0bfcdc303d5228c5a51d68f526e/third_party/WebKit/Source/core/paint/PaintLayerTest.cpp
,
Jan 12 2017
Tested the issue on windows 7, Linux Ubuntu 14.04 and Mac 10.12.1 using chrome version 57.0.2979.0.Not observed crash whlie navigating to URL https://davidwalsh.name/webgl-demo. Adding TE-Verified labels. Thanks,
,
Jan 23 2017
Issue 684058 has been merged into this issue.
,
Jan 23 2017
Issue 680649 has been merged into this issue.
,
Jan 30 2017
The crashes are not reproducible in Chrome OS 9202.10.0, 57.0.2987.17.
,
Feb 22 2017
Issue 681526 has been merged into this issue. |
||||||||||
►
Sign in to add a comment |
||||||||||
Comment 1 by rbasuvula@chromium.org
, Jan 10 2017Owner: chrishtr@chromium.org
Status: Assigned (was: Unconfirmed)