Reproducible crash on arstechnica [@ core::option::expect_failed | webrender::spatial_tree::SpatialTree::get_relative_transform_with_face ]
Categories
(Core :: Graphics: WebRender, defect, P2)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox-esr78 | --- | unaffected |
| firefox87 | --- | unaffected |
| firefox88 | + | fixed |
| firefox89 | + | fixed |
People
(Reporter: tnikkel, Assigned: gw)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details |
Using these steps I was able to bisect:
- make sure you don't have any arstechnica cookies.
- make sure you get the cookie dialog when opening arstechnica stories, for me this meant using a paris server on a vpn because they don't seem to show the cookie dialog in north america.
- open https://arstechnica.com/gadgets/2021/03/the-fairphone-2-hits-five-years-of-updates-with-some-help-from-lineageos/
- click "Show Purposes" on the cookie dialog
- try to scroll in the dialog
Note that I tested these steps in a nightly build with all of the display list building asserts as fatal from bug 1700812 (which are all of the asserts that I've come into when looking at the recent ASR problems) and we still hit the webrender crash and don't seem to hit the display list building asserts. (It's certainly still possible that the display list has an error in it that causes webrender to hit a problem but we just avoid the existing asserts somehow.)
|
Reporter | |
Updated•4 years ago
|
|
||
Updated•4 years ago
|
|
|
Reporter | |
Comment 1•4 years ago
|
||
(snip)
|
|
Reporter | |
Comment 2•4 years ago
|
||
Confirmed with a local debug build, I hit the crash in the parent process and do not hit any display list building asserts.
|
|
Reporter | |
Comment 3•4 years ago
|
||
[Tracking Requested - why for this release]:
I think the tracking 89 flag should be moved from bug 1695957 to this bug.
|
Assignee | |
Comment 4•4 years ago
|
||
I was able to reproduce this with a VPN connection and get a WR capture that reproduces the panic in wrench.
|
|
Assignee | |
Comment 5•4 years ago
|
||
|
||
Updated•4 years ago
|
|
|
||
Updated•4 years ago
|
|
||
Updated•4 years ago
|
|
||
Updated•4 years ago
|
|
||
Comment 7•4 years ago
|
||
| bugherder | ||
|
|
||
Comment 8•4 years ago
|
||
Please nominate this for Beta approval when you get a chance.
|
|
Assignee | |
Comment 9•4 years ago
|
||
Comment on attachment 9212092 [details]
Bug 1701361 - Fix nested scroll roots being selected for blend containers.
Beta/Release Uplift Approval Request
- User impact if declined: Reproducible GPU process panic on some content
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Small patch, fixes a clear panic, easy to back out of beta if any further problems are caused.
- String changes made/needed:
|
|
||
Comment 10•4 years ago
|
||
Comment on attachment 9212092 [details]
Bug 1701361 - Fix nested scroll roots being selected for blend containers.
Approved for 88.0b6.
|
|
||
Comment 11•4 years ago
|
||
| bugherder uplift | ||
|
||
Updated•4 years ago
|
Description
•