Closed
Bug 1709452
Opened 3 years ago
Closed 3 years ago
Do not store display list building rect in frames with continuations
Categories
(Core :: Web Painting, defect, P2)
Core
Web Painting
Tracking
()
RESOLVED
FIXED
90 Branch
People
(Reporter: mikokm, Assigned: mikokm)
References
Details
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
In bug 1707170 we uncovered an issue where retained display lists trip with frame continuations. If a modified frame is in the child list of the earlier unmodified continuation frame of the split frame, we cannot store display list building rect there, because we might not descend into that subframe tree.
Assignee | ||
Comment 1•3 years ago
|
||
Pushed by mikokm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/131f7f15056f Do not store display list building rect in frames with continuations r=mattwoodrow
Comment 3•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox90:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch
Assignee | ||
Comment 4•3 years ago
|
||
Comment on attachment 9220227 [details]
Bug 1709452 - Do not store display list building rect in frames with continuations r=mattwoodrow
Beta/Release Uplift Approval Request
- User impact if declined: Possible invalidation issues: style changes might not update correctly.
- 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): The patch is simple: it only adds an additional limitation where display list rebuild rectangles can be stored. This trades some performance for correctness.
- String changes made/needed:
Attachment #9220227 -
Flags: approval-mozilla-beta?
Comment 5•3 years ago
|
||
Comment on attachment 9220227 [details]
Bug 1709452 - Do not store display list building rect in frames with continuations r=mattwoodrow
Approved for 89.0b14.
Attachment #9220227 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 6•3 years ago
|
||
bugherder uplift |
status-firefox89:
--- → fixed
Flags: in-testsuite+
You need to log in
before you can comment on or make changes to this bug.
Description
•