Closed
Bug 1659850
Opened 5 years ago
Closed 5 years ago
Release display when we don't have a surface
Categories
(GeckoView :: General, enhancement, P3)
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: agi, Assigned: agi)
References
Details
Attachments
(1 file, 3 obsolete files)
If the app is in the background GeckoView is not visibile, and we shouldn't paint anything or run the compositor at all.
|
Assignee | |
Comment 1•5 years ago
|
||
Discussing with :snorp and :csadilek we think we shouldn't keep the Display when we don't have a surface, to make some "display already acquired" crashes go away. I'm morphing this bug to be that since it's related.
Summary: Release display when app goes to the background → Release display when we don't have a surface
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
||
Updated•5 years ago
|
Assignee: nobody → agi
Status: NEW → ASSIGNED
|
|
Assignee | |
Comment 3•5 years ago
|
||
|
|
Assignee | |
Comment 4•5 years ago
|
||
|
|
Assignee | |
Comment 5•5 years ago
|
||
|
|
||
Updated•5 years ago
|
Attachment #9171480 -
Attachment is obsolete: true
|
|
||
Updated•5 years ago
|
Attachment #9171479 -
Attachment is obsolete: true
|
|
||
Updated•5 years ago
|
Attachment #9171478 -
Attachment is obsolete: true
|
||
Updated•5 years ago
|
Priority: -- → P3
The current thinking is that we don't want to do this. We believe the exception in Bug 1651096 is valid, and a-c is unknowingly attaching multiple views to a single session.
|
|
Assignee | |
Updated•5 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•