Closed Bug 1292184 Opened 4 years ago Closed 4 years ago

remove lazy loading from inspector

Categories

(DevTools :: Framework, enhancement, P1)

enhancement

Tracking

(firefox51 fixed)

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

People

(Reporter: tromey, Assigned: tromey)

References

Details

(Whiteboard: [reserve-html])

Attachments

(1 file)

Investigation in bug 1278625 showed that lazy requires in the inspector
generally don't do anything -- the referenced modules are all loaded
elsewhere during inspector startup.  So, let's remove these.
After this patch there will still be some uses of lazyGetter et al.
However, these are all slated for removal by other de-chrome-ification bugs;
so I didn't bother touching them here.
Iteration: --- → 51.1 - Aug 15
Flags: qe-verify-
Priority: -- → P1
Whiteboard: [devtools-html] → [reserve-html]
Comment on attachment 8777850 [details]
Bug 1292184 - remove lazy requires from inspector;

https://reviewboard.mozilla.org/r/69296/#review66462

Nice, this looks good
Attachment #8777850 - Flags: review?(jlong) → review+
Pushed by ttromey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aaa21719d503
remove lazy requires from inspector; r=jlongster
https://hg.mozilla.org/mozilla-central/rev/aaa21719d503
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
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.