Closed
Bug 1234800
Opened 9 years ago
Closed 9 years ago
Bug 1147673 inadvertently removed dirty rect adjustment of fixed-pos frames
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla46
Tracking | Status | |
---|---|---|
firefox46 | --- | fixed |
People
(Reporter: mstange, Assigned: mstange)
References
Details
Attachments
(2 files)
2.14 KB,
patch
|
Details | Diff | Splinter Review | |
58 bytes,
text/x-review-board-request
|
tnikkel
:
review+
|
Details |
I only meant to remove the clip adjustment code, not the code that adjusts dirty rects.
Assignee | ||
Comment 1•9 years ago
|
||
I tried to write a test for this, but this test passes even before I reinstate the dirty rect adjustment. (It only passes with --e10s --setpref layout.scroll.root-frame-containers=true .)
Assignee: nobody → mstange
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/28961/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/28961/
Attachment #8701440 -
Flags: review?(tnikkel)
Comment 3•9 years ago
|
||
Comment on attachment 8701440 [details] MozReview Request: Bug 1234800 - Move this line to the right place. r?tn https://reviewboard.mozilla.org/r/28961/#review25749 Thanks!
Attachment #8701440 -
Flags: review?(tnikkel) → review+
Comment 5•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/e32159281b0c
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Assignee | ||
Comment 6•9 years ago
|
||
https://reviewboard.mozilla.org/r/28961/#review26683 ::: layout/base/nsDisplayList.cpp:750 (Diff revision 1) > nsRect dirty = dirtyRectRelativeToDirtyFrame - aFrame->GetOffsetTo(aDirtyFrame); Welp, this line needs to move down under the if. As it stands, this patch is completely useless.
Assignee | ||
Comment 7•9 years ago
|
||
Comment on attachment 8701440 [details] MozReview Request: Bug 1234800 - Move this line to the right place. r?tn Review request updated; see interdiff: https://reviewboard.mozilla.org/r/28961/diff/1-2/
Attachment #8701440 -
Attachment description: MozReview Request: Bug 1234800 - Reinstate code that adjusts dirty rects for fixed-position frames in display ports. r?tn → MozReview Request: Bug 1234800 - Move this line to the right place. r?tn
Comment 9•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/9edbc91c5b62
You need to log in
before you can comment on or make changes to this bug.
Description
•