Closed Bug 1402204 Opened 5 years ago Closed 5 years ago

Remove NS_SCROLLFRAME_INVALIDATE_CONTENTS_ON_SCROLL

Categories

(Core :: Web Painting, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: mattwoodrow, Assigned: mattwoodrow)

References

Details

Attachments

(1 file)

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 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
https://hg.mozilla.org/mozilla-central/rev/5d1b6657fdb0
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.