1611751 - Remove some nsIHTMLEditor, nsIEditorStyleSheets and nsITableEditor methods which are used by neither m-c, c-c nor BlueGriffon

Status: RESOLVED FIXED

(Core :: DOM: Editor, task, P5)

Description

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

+++ This bug was initially created as a clone of Bug #1610750 +++

Obviously, the following methods are not used by Gecko embedded products (I'm not sure about XUL addons for Thunderbird and SeaMonkey though).

• nsIHTMLEditor.removeAllInlineProperties c-c BG
• nsIHTMLEditor.increaseFontSize c-c BG
• nsIHTMLEditor.decreaseFontSize c-c BG
• nsIHTMLEditor.setParagraphFormat c-c BG
• nsIHTMLEditor.getBackgroundColorState c-c BG (Used in C++ only only by m-c)
• nsIHTMLEditor.indent c-c BG
• nsIHTMLEditor.align c-c BG
• nsIEditorStyleSheets.replaceOverrideStyleSheet c-c BG
• nsITableEditor.selectBlockOfCells c-c BG

We should just remove them from these interfaces.

Comment 1

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

Attachment: Bug 1611751 - Remove unused scriptable methods of `nsIHTMLEditor`, `nsIEditorStyleSheets` and `nsITableEditor` r=m_kato!

• nsIHTMLEditor.removeAllInlineProperties
• nsIHTMLEditor.increaseFontSize
• nsIHTMLEditor.decreaseFontSize
• nsIHTMLEditor.setParagraphFormat
• nsIHTMLEditor.getBackgroundColorState
• nsIHTMLEditor.indent
• nsIHTMLEditor.align
• nsIEditorStyleSheets.replaceOverrideStyleSheet
• nsITableEditor.selectBlockOfCells

These methods are not used by any Gecko products including comm-central and
BlueGriffon so that we should remove them. Note that only
HTMLEditor::GetBackgroundColorState() is used internally so that we need to
keep it as a public method of HTMLEditor.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b6856377aa69
Remove unused scriptable methods of `nsIHTMLEditor`, `nsIEditorStyleSheets` and `nsITableEditor` r=m_kato

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/b6856377aa69