Bug 1632682 Comment 13 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
I tested my changes on desktop locally by taking a profile:
- before my change: https://share.firefox.dev/3BGrJ0f
- after: https://share.firefox.dev/3IggiyS

Desktop appears to have the same pattern (`TelemetryControllerContent.jsm` loads before the ScriptLoader is init which presumably prevents it from being cached) which appears to be addressed by my change. Curiously, `IPCOut — PScriptCache::Msg___delete__` is delayed after my patch, letting us cache more scripts, but maybe that was a fluke.
Revision 1 by
on 
I tested my changes on desktop locally by making it sure it can load pages and inspecting profiles:
- before my change: https://share.firefox.dev/3BGrJ0f
- after: https://share.firefox.dev/3IggiyS

Desktop appears to have the same pattern (`TelemetryControllerContent.jsm` loads before the ScriptLoader is init which presumably prevents it from being cached) which appears to be addressed by my change. Curiously, `IPCOut — PScriptCache::Msg___delete__` is delayed after my patch, letting us cache more scripts, but maybe that was a fluke.

Back to Bug 1632682 Comment 13