|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
This causes a non-trivial startup performance hit.
Comment on attachment 8829625 [details] Bug 1333201: Don't import gDevTools.jsm before necessary. https://reviewboard.mozilla.org/r/106646/#review107842 r+ Thanks, I agree, it makes completely sense to don't import anything from the devtools internals until a devtools_page is actually used (and I'm going to take a second look to the other patches related to the devtools to be sure that we are going to do the same where needed). We should also create an uplift request for this patch, this part is already landed in Firefox 53 and it would be better to fix this there too.
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/386d5a9d293f Don't import gDevTools.jsm before necessary. r=rpl
Comment on attachment 8829625 [details] Bug 1333201: Don't import gDevTools.jsm before necessary. Approval Request Comment [Feature/Bug causing the regression]: Bug 1291737. [User impact if declined]: This adds non-trivial performance overhead to startup when loading WebExtensions that don't use the devtools API. [Is this code covered by automated tests?]: Yes. [Has the fix been verified in Nightly?]: N/A [Needs manual test from QE? If yes, steps to reproduce]: N/A [List of other uplifts needed for the feature/fix]: None. [Is the change risky?]: Very low-risk. [Why is the change risky/not risky?]: This change simply delays running certain initialization code until it's necessary. [String changes made/needed]: None.
I had to back this out for bustage like https://treeherder.mozilla.org/logviewer.html#?job_id=71613738&repo=autoland https://hg.mozilla.org/integration/autoland/rev/37ab612ff969708fe1c18c89bcfc387452343db6
https://hg.mozilla.org/integration/mozilla-inbound/rev/d7ad91f903facea2a0ff088c14d6ca4a501fb509 Bug 1333201: Don't import gDevTools.jsm before necessary. r=rpl
11 months ago
Comment on attachment 8829625 [details] Bug 1333201: Don't import gDevTools.jsm before necessary. Fix a startup perf. Aurora53+.