Closed
Bug 1459714
Opened 7 years ago
Closed 7 years ago
Don't delete compositables in an an earlier transaction than the one that removes references to them from the display list
Categories
(Core :: Graphics: WebRender, defect)
Tracking
()
RESOLVED
FIXED
mozilla62
Tracking | Status | |
---|---|---|
firefox62 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
(Whiteboard: [gfx-noted])
Attachments
(2 files)
After my fixes for bug 1459686, there are still errors with "Unknown pipline used for iframe blah blah blah" that I can reproduce while running reftests. Investigation showed that we're deleting the pipeline at [1] which happens just before the SetDisplayList call at [2] but in a different transaction. So at the time we delete the pipeline, the display list that WR has for the content is still referencing those pipelines, which is why we get this error.
[1] https://searchfox.org/mozilla-central/rev/b28b94dc81d60c6d9164315adbd4a5073526d372/gfx/layers/wr/WebRenderBridgeParent.cpp#767
[2] https://searchfox.org/mozilla-central/rev/b28b94dc81d60c6d9164315adbd4a5073526d372/gfx/layers/wr/WebRenderBridgeParent.cpp#655
Assignee | ||
Comment 1•7 years ago
|
||
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 4•7 years ago
|
||
mozreview-review |
Comment on attachment 8973878 [details]
Bug 1459714 - Make WebRenderBridgeParent::ProcessWebRenderParentCommands callers provide the transaction.
https://reviewboard.mozilla.org/r/242240/#review248136
Attachment #8973878 -
Flags: review?(sotaro.ikeda.g) → review+
Comment 5•7 years ago
|
||
mozreview-review |
Comment on attachment 8973879 [details]
Bug 1459714 - Use a single transaction for all things associated with a new display list.
https://reviewboard.mozilla.org/r/242242/#review248138
Attachment #8973879 -
Flags: review?(sotaro.ikeda.g) → review+
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Pushed by kgupta@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/af3758713a44
Make WebRenderBridgeParent::ProcessWebRenderParentCommands callers provide the transaction. r=sotaro
https://hg.mozilla.org/integration/autoland/rev/7b3c982608df
Use a single transaction for all things associated with a new display list. r=sotaro
Comment 9•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/af3758713a44
https://hg.mozilla.org/mozilla-central/rev/7b3c982608df
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox62:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in
before you can comment on or make changes to this bug.
Description
•