Closed
Bug 1272801
Opened 9 years ago
Closed 9 years ago
PaintedLayerDataTree::FindPaintedLayerFor has an argument aForceOwnLayer that's always false
Categories
(Core :: Layout, defect)
Core
Layout
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
|
Assignee | |
Comment 1•9 years ago
|
||
(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++]
|
|
Assignee | |
Comment 2•9 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/57330/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/57330/
Attachment #8759339 -
Flags: review?(matt.woodrow)
|
||
Comment 3•9 years ago
|
||
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
|
||
Comment 5•9 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in
before you can comment on or make changes to this bug.
Description
•