Closed
Bug 1408127
Opened 8 years ago
Closed 8 years ago
Integrate a pre-processor for user scripts
Categories
(Firefox for iOS :: General, enhancement, P2)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| fxios | 11.0 | --- |
People
(Reporter: justindarc, Assigned: justindarc)
References
Details
Attachments
(1 file)
To improve developer ergonomics and webview performance, our user scripts should be concatenated and minified using a JavaScript pre-processor. This would allow us to use ES7 modules for code reusability while drastically reducing the overall number and size of scripts injected into the webview.
At an initial glance, it seems that we need to consider at least 4 groupings of user scripts:
- onDocumentStart (per frame)
- onDocumentStart (per window)
- onDocumentEnd (per frame)
- onDocumentEnd (per window)
In order to make integration seamless from a developer's standpoint, I propose that we categorize our user scripts into separate folders in Xcode based on the 4 groupings above. This would allow us to simply hook our pre-processor into a Build Phase in Xcode so that these scripts are compiled automatically at build-time.
| Assignee | ||
Comment 1•8 years ago
|
||
Assigning to myself for v11.0 as per discussion in today's TDES meeting. Marking as P2 for now until I begin actively working on this.
| Assignee | ||
Comment 2•8 years ago
|
||
Attachment #8943355 -
Flags: review?(gkeeley)
Comment on attachment 8943355 [details] [review]
GitHub Pull Request
r+
I am following up with a BB validation step in https://bugzilla.mozilla.org/show_bug.cgi?id=1431751 and I'll ping you for review
Attachment #8943355 -
Flags: review?(gkeeley) → review+
| Assignee | ||
Comment 4•8 years ago
|
||
Landed on master:
https://github.com/mozilla-mobile/firefox-ios/commit/2d5ca06880b9288be74b7eb1a6cb7c937292387d
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•