Closed Bug 1394583 Opened 2 years ago Closed 2 years ago

Remove default properties from nsIHTMLEditor

Categories

(Core :: Editor, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: njn, Assigned: njn)

Details

Attachments

(1 file)

masayuki, I'm not sure if this is a reasonable thing to do, but a try server
push works fine!
nsIHTMLEditor's addDefaultProperty(), removeDefaultProperty(),
removeAllDefaultProperties() methods are never used -- not from C++ code or
script, in either mozilla-central or comm-central.

So this patch removes them. This means that HMLTEditor::mDefaultStyles is never
used, so the patch removes it and all the code that manipulates it as well.
Attachment #8902004 - Flags: review?(masayuki)
Comment on attachment 8902004 [details] [diff] [review]
Remove default properties from nsIHTMLEditor

Really nice!
Attachment #8902004 - Flags: review?(masayuki) → review+
https://hg.mozilla.org/mozilla-central/rev/5d8fdaa9efc8
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
You need to log in before you can comment on or make changes to this bug.