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)

defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: mstange, Unassigned)

References

Details

Attachments

(1 file)

Attached patch remove it (deleted) — Splinter Review
See bug 926292 comment 33.
Attachment #8430010 - Flags: review?(tnikkel)
That comment says probably! :) I'll have to think this through to make sure it's not having a positive in any case anymore.
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.
Comment on attachment 8430010 [details] [diff] [review]
remove it

Good points, I agree.
Attachment #8430010 - Flags: review?(tnikkel)
Assignee: mstange → nobody
Status: ASSIGNED → NEW
Depends on: multi-layer-apz
Looks like the IsScrollLayerItemAndOverlayScrollbarForScrollFrame function still exists, does it still need removing?
Flags: needinfo?(mstange)
Flags: needinfo?(mstange) → needinfo?(tnikkel)
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)
Pretty sure this is gone now.
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.

Attachment

General

Created:
Updated:
Size: