Closed Bug 1476906 Opened Last year Closed Last year

Remove ReplaceHeadContentsWithHTML() from nsIHTMLEditor

Categories

(Core :: DOM: Editor, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

References

Details

Attachments

(1 file)

There is no users of ReplaceHeadContentsWithHTML() outside of editor. So, we can make this method just a non-virtual method.
Comment on attachment 8993616 [details]
Bug 1476906 - Change nsIHTMLEditor::ReplaceHeadContentsWithHTML() to a non-virtual method of HTMLEditor

https://reviewboard.mozilla.org/r/258086/#review265628
Attachment #8993616 - Flags: review?(m_kato) → review+
Pushed by masayuki@d-toybox.com:
https://hg.mozilla.org/integration/autoland/rev/bd51af94b408
Change nsIHTMLEditor::ReplaceHeadContentsWithHTML() to a non-virtual method of HTMLEditor r=m_kato
https://hg.mozilla.org/mozilla-central/rev/bd51af94b408
Status: ASSIGNED → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
(In reply to Masayuki Nakano [:masayuki] (JST, +0900) from comment #0)
> There is no users of ReplaceHeadContentsWithHTML() outside of editor. So, we
> can make this method just a non-virtual method.
I've just noticed: My add-on ThunderHTMLedit was using that function :-(
You need to log in before you can comment on or make changes to this bug.