Closed Bug 1491191 Opened Last year Closed Last year
Remove unused methods in ns
IEditor Mail Support
46 bytes, text/x-phabricator-request
|Details | Review|
- 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.
s/rewrap is unsed by comm-central/rewrap is used by comm-central/
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 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.
Attachment #9008993 - Flags: review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/0b212a6e641a Remove unused methods in nsIEditorMailSupport. r=masayuki
You need to log in before you can comment on or make changes to this bug.