Closed
Bug 1259777
Opened 8 years ago
Closed 8 years ago
DOMUtils is (lazy) loaded but not used in css-angle.js
Categories
(DevTools :: General, defect, P2)
DevTools
General
Tracking
(firefox48 fixed)
RESOLVED
FIXED
Firefox 48
Tracking | Status | |
---|---|---|
firefox48 | --- | fixed |
People
(Reporter: pbro, Assigned: nchevobbe)
Details
(Whiteboard: [btpp-fix-later])
Attachments
(1 file)
In the file devtools/shared/css-angle.js, there's a lazy getter at the bottom that access DOMUtils, but DOMUtils isn't used anywhere in the file.
Assignee | ||
Comment 1•8 years ago
|
||
Review commit: https://reviewboard.mozilla.org/r/42487/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/42487/
Attachment #8734799 -
Flags: review?(pbrosset)
Reporter | ||
Comment 2•8 years ago
|
||
Comment on attachment 8734799 [details] MozReview Request: Bug 1259777 - Remove unnecessary DOMUtils lazy load in css-angle.js . r=pbro https://reviewboard.mozilla.org/r/42487/#review38963 Thanks for the quick fix.
Attachment #8734799 -
Flags: review?(pbrosset) → review+
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → chevobbe.nicolas
Status: NEW → ASSIGNED
Comment 4•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3e41c15a76b5
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox48:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•