Closed Bug 1491191 Opened Last year Closed Last year

Remove unused methods in nsIEditorMailSupport

Categories

(Core :: Editor, enhancement, P3)

enhancement

Tracking

()

RESOLVED FIXED
mozilla64
Tracking Status
firefox64 --- fixed

People

(Reporter: m_kato, Assigned: m_kato)

Details

Attachments

(1 file)

- 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 m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/0b212a6e641a
Remove unused methods in nsIEditorMailSupport. r=masayuki
https://hg.mozilla.org/mozilla-central/rev/0b212a6e641a
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.