1491191 - Remove unused methods in nsIEditorMailSupport

Status: RESOLVED FIXED

(Core :: DOM: Editor, enhancement, P3)

Description

[Makoto Kato [:m_kato]]

- insertAsCitedQuotation is used by test (editor/libeditor/tests/test_bug616590.xul).  But I don't know we should keep this for test.
- rewrap is unsed by comm-central.  Although it used by cmd_rewrap, we might have to move this to c-c or move command implementation to gecko's editor code, such as cmd_removeLinks...

Others are unused now.

Comment 1

[Makoto Kato [:m_kato]]

s/rewrap is unsed by comm-central/rewrap is used by comm-central/

Comment 2

[Makoto Kato [:m_kato]]

Attachment: Bug 1491191 - Remove unused methods in nsIEditorMailSupport. r?masayuki

Since I have landed bug 1489939, comm-central only uses rewrap method in
nsIEditorMailSupport.  And insertAsCitedQuotation is used by the test of
editor/libeditor/tests/test_bug616590.xul.

Other methods are unused now, so let's remove these from nsIEditorMailSupprt,
and move it to HTMLEditor.  Also, pasteAsQuotation is unused now even if
BlueGriffon.

Comment 3

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

Comment on attachment 9008993 [details]
Bug 1491191 - Remove unused methods in nsIEditorMailSupport. r?masayuki

Masayuki Nakano [:masayuki] (JST, +0900) (offline: 9/21-9/30) has approved the revision.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0b212a6e641a
Remove unused methods in nsIEditorMailSupport. r=masayuki

Comment 5

[Eliza Balazs [:ebalazs_]]

https://hg.mozilla.org/mozilla-central/rev/0b212a6e641a