1292184 - remove lazy loading from inspector

Status: RESOLVED FIXED

(DevTools :: Framework, enhancement, P1)

Description

[Tom Tromey :tromey]

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.

Comment 1

[Tom Tromey :tromey]

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.

Comment 2

[Tom Tromey :tromey]

Attachment: Bug 1292184 - remove lazy requires from inspector;

Review commit: https://reviewboard.mozilla.org/r/69296/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69296/

Comment 3

[James Long (:jlongster)]

Comment on attachment 8777850 [details]
Bug 1292184 - remove lazy requires from inspector;

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

Nice, this looks good

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/aaa21719d503
remove lazy requires from inspector; r=jlongster

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/aaa21719d503