Closed
Bug 1265515
Opened 8 years ago
Closed 8 years ago
Remove nsAutoPtr.h includes from gfx/layers/apz
Categories
(Core :: Panning and Zooming, defect)
Core
Panning and Zooming
Tracking
()
RESOLVED
FIXED
mozilla48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: botond, Assigned: botond)
Details
Attachments
(1 file)
In APZ code, we've historically included nsAutoPtr.h to get nsRefPtr. nsRefPtr has since been renamed to RefPtr, and is now defined in mfbt/RefPtr.h. The includes should be adjusted accordingly.
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/47237/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/47237/
Attachment #8742485 -
Flags: review?(bugmail.mozilla)
Comment 2•8 years ago
|
||
Comment on attachment 8742485 [details] MozReview Request: Bug 1265515 - Update nsAutoPtr.h includes in gfx/layers/apz to RefPtr.h. r=kats https://reviewboard.mozilla.org/r/47237/#review43893 LGTM, thanks!
Attachment #8742485 -
Flags: review?(bugmail.mozilla) → review+
Assignee | ||
Comment 3•8 years ago
|
||
Try push that includes this patch: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f53dd840d47c
Assignee | ||
Comment 4•8 years ago
|
||
Group-landed with bug 1266154.
Comment 6•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/d430b697c76d
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in
before you can comment on or make changes to this bug.
Description
•