Closed
Bug 1320628
Opened 6 years ago
Closed 6 years ago
Fix background tab crash when e10s is enabled
Categories
(Core :: Graphics: WebRender, defect)
Core
Graphics: WebRender
Tracking
()
RESOLVED
FIXED
mozilla54
People
(Reporter: sotaro, Assigned: sotaro)
References
Details
Attachments
(1 file, 4 obsolete files)
|
22.68 KB,
patch
|
Details | Diff | Splinter Review |
No description provided.
|
Assignee | |
Updated•6 years ago
|
Assignee: nobody → sotaro.ikeda.g
|
|
Assignee | |
Comment 1•6 years ago
|
||
|
|
Assignee | |
Comment 2•6 years ago
|
||
attachment 8814799 [details] [diff] [review] fixed background tab crash when e10s is enabled. But it caused a rendering problem when a background tab becomes foreground.
|
|
Assignee | |
Comment 3•6 years ago
|
||
LayerObserverEpoch might be related.
|
|
Assignee | |
Updated•6 years ago
|
Summary: Fix Assert failure of GetBackendType() in TabChild → Fix background tab crash when e10s is enabled
|
|
Assignee | |
Comment 4•6 years ago
|
||
Attachment #8814799 -
Attachment is obsolete: true
|
|
Assignee | |
Updated•6 years ago
|
Attachment #8815165 -
Flags: review?(nical.bugzilla)
|
|
Assignee | |
Updated•6 years ago
|
Attachment #8815165 -
Flags: review?(nical.bugzilla)
|
|
Assignee | |
Comment 5•6 years ago
|
||
Comment on attachment 8815165 [details] [diff] [review] patch - Fix background tab crash when e10s is enabled If we are going to add more support of AsWebRenderLayerManager(), we are going to see AsWebRenderLayerManager() and AsClientLayerManager() in a lot of places. :nical, what do you prefer to reduce them?
Attachment #8815165 -
Flags: feedback?(nical.bugzilla)
|
||
Comment 6•6 years ago
|
||
Comment on attachment 8815165 [details] [diff] [review] patch - Fix background tab crash when e10s is enabled Review of attachment 8815165 [details] [diff] [review]: ----------------------------------------------------------------- Could you try to expose the duplicated method into the base LayerManager class?
Attachment #8815165 -
Flags: feedback?(nical.bugzilla)
|
|
Assignee | |
Comment 7•6 years ago
|
||
Apply the comment.
Attachment #8815165 -
Attachment is obsolete: true
|
||
Comment 10•6 years ago
|
||
Pushed by sikeda@mozilla.com: https://hg.mozilla.org/projects/graphics/rev/e98fd107f5a0 Fix background tab crash when e10s is enabled r=nical?
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
|
||
Updated•6 years ago
|
Target Milestone: --- → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•