Closed Bug 1354328 Opened 3 years ago Closed 3 years ago

Lazily load inline spell checker

Categories

(Firefox :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 55
Tracking Status
firefox55 --- fixed

People

(Reporter: mccr8, Assigned: mccr8)

References

(Blocks 1 open bug)

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.)
Blocks: 1350472
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+
Pushed by amccreight@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7886d794b829
Lazily load the inline spell checker. r=Felipe
https://hg.mozilla.org/mozilla-central/rev/7886d794b829
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
You need to log in before you can comment on or make changes to this bug.