Closed
Bug 1402204
Opened 8 years ago
Closed 8 years ago
Remove NS_SCROLLFRAME_INVALIDATE_CONTENTS_ON_SCROLL
Categories
(Core :: Web Painting, enhancement)
Core
Web Painting
Tracking
()
RESOLVED
FIXED
mozilla58
| Tracking | Status | |
|---|---|---|
| firefox58 | --- | fixed |
People
(Reporter: mattwoodrow, Assigned: mattwoodrow)
References
Details
Attachments
(1 file)
|
5.19 KB,
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
I'm fairly sure we don't need this any more as DLBI should always correctly detect positions changing (relative to the scroll container) and invalidate them.
It seems to pass all tests, do you know of anything else I should manually?
Attachment #8911026 -
Flags: review?(mats)
|
||
Comment 1•8 years ago
|
||
Comment on attachment 8911026 [details] [diff] [review]
invalidate-on-scroll
I can't think of anything, so this looks fine to me. r=mats
(don't forget to add the bug number and reviewer to the commit message)
Attachment #8911026 -
Flags: review?(mats) → review+
Pushed by mwoodrow@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/5d1b6657fdb0
Remove NS_SCROLLFRAME_INVALIDATE_CONTENTS_ON_SCROLL as it doesn't appear to be needed any more. r=mats
|
||
Comment 3•8 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in
before you can comment on or make changes to this bug.
Description
•