Make editor stop exposing its `TransactionManager` to chrome script
Categories
(Core :: DOM: Editor, task, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox108 | --- | fixed |
People
(Reporter: masayuki, Assigned: masayuki)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
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®exp=false
Then, we could stop using nsITransactionManager and/or nsITransaction interface in editor to save the footprint.
|
Assignee | |
Updated•2 years ago
|
|
|
Assignee | |
Comment 1•2 years ago
|
||
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.
|
||
Updated•2 years ago
|
|
|
Assignee | |
Updated•2 years ago
|
Pushed by masayuki@d-toybox.com: https://hg.mozilla.org/integration/autoland/rev/fd01384fd33a Get rid of `nsIEditor.transactionManager` due to unused r=m_kato
|
||
Comment 3•2 years ago
|
||
| bugherder | ||
Description
•