1310385 - Remove support for UndoManager

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[(no longer active)]

There hasn't been any traction on this feature, and we keep
paying the maintenance and code size and memory usage penalty
of having this implementation.

We can revive this from VCS history in the future if we decide
to start working on it again.  In the mean time, it's better
to remove it.

Comment 1

[(no longer active)]

Attachment: Remove support for UndoManager

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8801414 [details] [diff] [review]
Remove support for UndoManager

-    "DOMTransactionEvent",
-// IMPORTANT: Do not change this list without review from a DOM peer!

oh crap, we've exposed some of this stuff to the web :(

Comment 3

[(no longer active)]

(In reply to Olli Pettay [:smaug] from comment #2)
> Comment on attachment 8801414 [details] [diff] [review]
> Remove support for UndoManager
> 
> -    "DOMTransactionEvent",
> -// IMPORTANT: Do not change this list without review from a DOM peer!
> 
> oh crap, we've exposed some of this stuff to the web :(

Sadly yes.  :(

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/fa17bd43674a
Remove support for UndoManager; r=smaug

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

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

Comment 6

[Kohei Yoshino [:kohei]]

MDN apparently has some mentions of the API, e.g. https://developer.mozilla.org/en-US/docs/Web/API/Element

No site-compat effect since it's disabled by default.