Closed Bug 1310385 Opened 3 years ago Closed 3 years ago
Remove support for Undo
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 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 :(
Attachment #8801414 - Flags: review?(bugs) → review+
(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. :(
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/fa17bd43674a Remove support for UndoManager; r=smaug
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.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.