Closed
Bug 1354328
Opened 7 years ago
Closed 7 years ago
Lazily load inline spell checker
Categories
(Firefox :: General, defect)
Firefox
General
Tracking
()
RESOLVED
FIXED
Firefox 55
| Tracking | Status | |
|---|---|---|
| firefox55 | --- | fixed |
People
(Reporter: mccr8, Assigned: mccr8)
References
Details
Attachments
(1 file)
Gijs pointed out this is easy to avoid loading these .jsms. You just have to make InlineSpellChecker.jsm and InlineSpellCheckerContent.jsm loaded lazily in content.js. Then they will only get loaded if you open a context menu. (The former is needed for SpellCheckHelper.)
|
Assignee | |
Comment 1•7 years ago
|
||
L64 try looks good: https://treeherder.mozilla.org/#/jobs?repo=try&revision=0b3038140466797cf9ffe087b293966a6ba625d9
| Comment hidden (mozreview-request) |
|
||
Comment 3•7 years ago
|
||
| mozreview-review | ||
Comment on attachment 8855926 [details] Bug 1354328 - Lazily load the inline spell checker. https://reviewboard.mozilla.org/r/127812/#review130502
Attachment #8855926 -
Flags: review?(felipc) → review+
| Comment hidden (obsolete) |
| Comment hidden (mozreview-request) |
Pushed by amccreight@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7886d794b829 Lazily load the inline spell checker. r=Felipe
|
||
Comment 7•7 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/7886d794b829
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in
before you can comment on or make changes to this bug.
Description
•