Closed Bug 1491274 Opened 5 years ago Closed 5 years ago
Split content script-only user
Scripts API from ext-user Scripts .js and user _scripts .json
The schemas and implementations of the content-script-side userScripts API (introduced in bug 1437864) and the add-on side userScripts API are fully independent. The implementation of the content-script only logic is just a few lines, whereas the extension process part of the API spans over 100 lines. Since we put so much efforts into minimizing content proces memory usage elsewhere, we should split these in separate files (user_scripts.json and ext-userScript.js). I've done something similar before when I introduced the menus.getTargetElement API, so I'll take this bug.
Comment on attachment 9009968 [details] Bug 1491274 - Split content script-only userScripts API from ext-userScripts.js and user_scripts.json. r?mixedpuppy!,robwu Shane Caraveo (:mixedpuppy) has approved the revision.
Hey Rob, I'm taking this and added you as an additional reviewer on the attached patch (I've been already working on other changes related to these files, and so I thought that it was worth to do it right now).
Assignee: rob → lgreco
Iteration: --- → 64.2 (Sep 28)
Priority: -- → P1
Comment on attachment 9009968 [details] Bug 1491274 - Split content script-only userScripts API from ext-userScripts.js and user_scripts.json. r?mixedpuppy!,robwu Rob Wu [:robwu] has approved the revision.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/d2e7433921cf Split content script-only userScripts API from ext-userScripts.js and user_scripts.json. r=robwu,mixedpuppy
This is tricky to QA verify manually (which of the API modules are loaded or not loaded in the various Firefox processes), but we could at least verify that the change didn't regressed any of the tests from the list of test cases mentioned in Bug 1437861 comment 44.
No regressions found after running all cases mentioned in https://bugzilla.mozilla.org/show_bug.cgi?id=1437861#c44 . Marking bug as verified.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.