Closed
Bug 1311572
Opened 8 years ago
Closed 8 years ago
Remove code associated with comments linking to bug 1265759
Categories
(DevTools :: General, defect)
DevTools
General
Tracking
(firefox52 fixed)
RESOLVED
FIXED
Firefox 52
| Tracking | Status | |
|---|---|---|
| firefox52 | --- | fixed |
People
(Reporter: ntim, Assigned: gasolin)
References
Details
Attachments
(1 file)
There are some comments linking to bug 1265759. Now that bug 1265759 is fixed, we can remove the comments and the code related to it.
https://dxr.mozilla.org/mozilla-central/source/devtools/client/themes/inspector.css#117
https://dxr.mozilla.org/mozilla-central/source/devtools/client/themes/inspector.css#108
| Comment hidden (mozreview-request) |
|
Assignee | |
Comment 3•8 years ago
|
||
Removed styles and visually spot no regression
|
Reporter | |
Comment 4•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8802780 [details]
Bug 1311572 - Remove code associated with comments linking to bug 1265759;
https://reviewboard.mozilla.org/r/87076/#review86436
Thanks! Looks good to me.
Attachment #8802780 -
Flags: review?(ntim.bugs) → review+
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a738ff48cc9e
Remove code associated with comments linking to bug 1265759;r=ntim
Keywords: checkin-needed
|
||
Comment 7•8 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox52:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 52
|
||
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•