Closed Bug 1272801 Opened 3 years ago Closed 3 years ago

PaintedLayerDataTree::FindPaintedLayerFor has an argument aForceOwnLayer that's always false

Categories

(Core :: Layout, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla49
Tracking Status
firefox49 --- fixed

People

(Reporter: mstange, Assigned: mstange)

References

Details

Attachments

(1 file)

This argument is no longer needed, the only caller passes "false" always. We should remove the argument and anything else that becomes unnecessary because of it.

https://dxr.mozilla.org/mozilla-central/rev/1f1a8b96d5167153d1f750439ba6a1063155a4bc/layout/base/FrameLayerBuilder.cpp#2822
(Apparently I should have put my email address into the "Mentors" field, not into the whiteboard.)
Assignee: nobody → mstange
Status: NEW → ASSIGNED
Whiteboard: [good first bug][mentor=mstange][lang=c++]
Comment on attachment 8759339 [details]
Bug 1272801 - Remove unused aForceOwnLayer argument.

https://reviewboard.mozilla.org/r/57330/#review54148
Attachment #8759339 - Flags: review?(matt.woodrow) → review+
Pushed by mstange@themasta.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b28d1c65f0a1
Remove unused aForceOwnLayer argument. r=mattwoodrow
https://hg.mozilla.org/mozilla-central/rev/b28d1c65f0a1
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in before you can comment on or make changes to this bug.