Closed
Bug 1139153
Opened 10 years ago
Closed 10 years ago
Remove the layout.async-containerless-scrolling.enabled pref
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla39
Tracking | Status | |
---|---|---|
firefox39 | --- | fixed |
People
(Reporter: kats, Assigned: kats)
References
Details
Attachments
(1 file)
10.67 KB,
patch
|
tnikkel
:
review+
|
Details | Diff | Splinter Review |
The layout.async-containerless-scrolling.enabled pref is true everywhere and can be removed.
Assignee | ||
Comment 1•10 years ago
|
||
I'll do a try push before landing, but try is closed right now.
Attachment #8572186 -
Flags: review?(tnikkel)
Comment 2•10 years ago
|
||
(In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #0)
> The layout.async-containerless-scrolling.enabled pref is true everywhere and
> can be removed.
Congratulations! \o/
Comment 3•10 years ago
|
||
Comment on attachment 8572186 [details] [diff] [review]
Patch
Hooray. There's even more cleanup that this enables, but it's not straight code deletion. nsDisplayScrollLayer items are no longer created, and we shouldn't need ScrollLayerWrapper at all anymore. But those are a bit more complicated, so let's land this without delay.
Attachment #8572186 -
Flags: review?(tnikkel) → review+
Assignee | ||
Comment 4•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 6•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-firefox39:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
Updated•10 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•