Get rid of nsISpellChecker

RESOLVED FIXED in Firefox 65

Status

()

enhancement
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: m_kato, Assigned: m_kato)

Tracking

unspecified
mozilla65
Points:
---

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(2 attachments)

nsISpellChecker isn't scriptable, and mozSpellChecker is only implementation.  So we can get rid of nsISpellChecker
Assignee: nobody → m_kato
When creating an instance of nsISpellChecker, we always use
mozSpellChecker::Create.  So we should use mozSpellChecker directly instead of
nsISpellChecker.
No one uses nsISpellChecker, so let's get rid of nsISpellChecker.

Depends on D10993
Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/ef65164fcac0
Part 1. Use mozSpellChecker directly instead of nsISpellChecker. r=masayuki
https://hg.mozilla.org/integration/autoland/rev/89e02e0a1b77
Part 2. Get rid of nsISpellChecker. r=masayuki
Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6a089fb17fe6
Part 1. Use mozSpellChecker directly instead of nsISpellChecker. r=masayuki
https://hg.mozilla.org/integration/mozilla-inbound/rev/b3888d4eeaa3
Part 2. Get rid of nsISpellChecker. r=masayuki
Flags: needinfo?(m_kato)
(In reply to Jorg K (GMT+1) from comment #6)
> This doesn't affect TB, right?
> https://searchfox.org/comm-central/
> search?q=GetNextMisspelledWord&case=false&regexp=false&path=
> https://searchfox.org/comm-central/rev/
> 105e71c4fd8d65f428890a4c562f9f687c2566e8/editor/ui/dialogs/content/
> EdSpellCheck.js#26

nsISpellChecker isn't scriptable.  It is nsIEditorSpellCheck
https://hg.mozilla.org/mozilla-central/rev/6a089fb17fe6
https://hg.mozilla.org/mozilla-central/rev/b3888d4eeaa3
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
You need to log in before you can comment on or make changes to this bug.