Closed
Bug 1016939
Opened 10 years ago
Closed 6 years ago
Back out bug 976370 because bug 926292 (with bug 987680) has made it obsolete
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: mstange, Unassigned)
References
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
See bug 926292 comment 33.
Attachment #8430010 -
Flags: review?(tnikkel)
Comment 1•10 years ago
|
||
That comment says probably! :) I'll have to think this through to make sure it's not having a positive in any case anymore.
Comment 2•10 years ago
|
||
I'm leaning towards just leaving this in until we rip it out wholesale with bug 967844. It doesn't cause any harm, and removing it we might regress something. Not really worth the effort to try to prove it's unneeded.
Reporter | ||
Comment 3•10 years ago
|
||
Comment on attachment 8430010 [details] [diff] [review] remove it Good points, I agree.
Attachment #8430010 -
Flags: review?(tnikkel)
Reporter | ||
Updated•10 years ago
|
Comment 4•10 years ago
|
||
Looks like the IsScrollLayerItemAndOverlayScrollbarForScrollFrame function still exists, does it still need removing?
Updated•10 years ago
|
Flags: needinfo?(mstange)
Reporter | ||
Updated•10 years ago
|
Flags: needinfo?(mstange) → needinfo?(tnikkel)
Comment 5•10 years ago
|
||
We should be able to remove this and a whole bunch of other code wholesale now that multilayer apz has landed and stuck.
Flags: needinfo?(tnikkel)
Comment 6•9 years ago
|
||
Pretty sure this is gone now.
Reporter | ||
Comment 7•6 years ago
|
||
Agreed.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•