Closed Bug 1292253 Opened 8 years ago Closed 8 years ago

remove chrome use from devtools/client/shared/widgets/view-helpers.js

Categories

(DevTools :: Framework, enhancement, P1)

Product:
DevTools
Component:
Framework
Type:
enhancement

Tracking

(firefox51 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 51
Iteration:
51.1 - Aug 15
Tracking Flags:
Tracking Status
firefox51 --- fixed

People

(Reporter: tromey, Assigned: tromey)

References

Details

(Whiteboard: [reserve-html])

Attachments

(1 file)

Bug 1292253 - replace some Ci uses in view-helpers and TableWidget;
58 bytes, text/x-review-board-request
gregtatum
: review+
Details
Inspector startup loads devtools/client/shared/widgets/view-helpers.js, which requires chrome. This require must be removed for the de-chrome-ification project.
Flags: qe-verify-
Priority: -- → P2
Whiteboard: [devtools-html] → [reserve-html]
Assignee: nobody → ttromey
Status: NEW → ASSIGNED
Iteration: --- → 51.1 - Aug 15
Priority: P2 → P1
Comment on attachment 8779394 [details] Bug 1292253 - replace some Ci uses in view-helpers and TableWidget; https://reviewboard.mozilla.org/r/70378/#review67814 This patch makes me inordinately happy.
Attachment #8779394 - Flags: review?(gtatum) → review+
Pushed by ttromey@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/3133dc006e46 replace some Ci uses in view-helpers and TableWidget; r=gregtatum
https://hg.mozilla.org/mozilla-central/rev/3133dc006e46
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox51: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 51
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: