1768315 - Make editor stop exposing its `TransactionManager` to chrome script

Status: RESOLVED FIXED

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

Description

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

It's currently used for clearing undo transactions. So, it's enough to make nsIEditor have alternative API.
https://searchfox.org/comm-central/search?q=.transactionManager&path=&case=true&regexp=false

Then, we could stop using nsITransactionManager and/or nsITransaction interface in editor to save the footprint.

Comment 1

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

Attachment: Bug 1768315 - Get rid of `nsIEditor.transactionManager` due to unused r=m_kato!

nsITransactionManager provides too low level things and editor may want to
stop using it in the future if Undo Manager spec is implemented in another
browser. Therefore, I'd like to stop exposing nsITransactionManager instance
for editor.

Note that mozilla::TransactionManager is still created and used by the
UI of mailer of SeaMonkey (i.e., under mailnews), once we move
nsITransactionManager instance moved into SeaMonkey, we can make
mozilla::TransactionManager stop inheriting nsITransactionManager.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/fd01384fd33a
Get rid of `nsIEditor.transactionManager` due to unused r=m_kato

Comment 3

[Cristian Tuns]

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