1337247 - Make pasteAsCitedQuotation as noscript

Status: RESOLVED FIXED

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

Description

[Makoto Kato [:m_kato]]

No addons and c-c's JS code doesn't use pasteAsCitedQuotation.  So I would like that this changes to noscript method.

Comment 1

[Jorg K (CEST = GMT+2)]

Fine by me ;-)

Comment 2

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(sickish, maybe slower respose)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c1a80561a23f81987811f20476186662e80cb4e5

Comment 3

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)(sickish, maybe slower respose)]

Attachment: Bug 1337247 - Remove PasteAsCitedQuotation() from nsIEditorMailSupport

nsIEditorMailSupport::PasteAsCitedQuotation() is only used by
HTMLEditor::PasteAsQuotation().  So, we can get rid of the method from the
interface and merge current implementation with HTMLEditor::PasteAsQuotation().

Review commit: https://reviewboard.mozilla.org/r/258044/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/258044/

Comment 4

[Makoto Kato [:m_kato]]

Comment on attachment 8993257 [details]
Bug 1337247 - Remove PasteAsCitedQuotation() from nsIEditorMailSupport

https://reviewboard.mozilla.org/r/258044/#review265326

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3c2db10f62de
Remove PasteAsCitedQuotation() from nsIEditorMailSupport r=m_kato

Comment 6

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/3c2db10f62de