1368189 - Move more code out of ExtensionUtils.jsm

Status: RESOLVED FIXED

(WebExtensions :: General, defect, P1)

Description

[Kris Maglione [:kmag]]

ExtensionUtils.jsm gets loaded fairly early in all processes, but a lot of the code there is not needed in all processes, or is not needed until extension code runs. Most of it should be moved to ExtensionCommon, ExtensionParent, or ExtensionChild.

Comment 1

[Kris Maglione [:kmag]]

Attachment: Bug 1368189: Move more code out of ExtensionUtils.jsm.

Also removes some dead code.

A lot of the code in ExtensionUtils.jsm is not needed in all processes, and a
lot of the rest isn't needed until extension code runs. Most of it winds up
being loaded into all processes way earlier than necessary.

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

Comment 2

[Shane Caraveo (:mixedpuppy)]

Comment on attachment 8871971 [details]
Bug 1368189: Move more code out of ExtensionUtils.jsm.

https://reviewboard.mozilla.org/r/143504/#review149308

Comment 3

[Kris Maglione [:kmag]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/92040a759ddd4c1496dc64e1eb9173805fe25516
Bug 1368189: Move more code out of ExtensionUtils.jsm. r=mixedpuppy

Comment 4

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/92040a759ddd