Closed Bug 1995912 Opened 1 year ago Closed 1 year ago

Get rid of tests under `testing/web-platform/editing` if they depend on the legacy DOM mutation events

Categories

(Core :: DOM: Editor, task)

task

Tracking

()

RESOLVED FIXED
146 Branch
Tracking Status
firefox146 --- fixed

People

(Reporter: masayuki, Assigned: masayuki)

References

Details

Attachments

(1 external file)

The original issue is here: https://github.com/web-platform-tests/wpt/issues/54682

Now, we can drop the tests to save the cost in CI of all browser vendors.

Assignee: nobody → masayuki
Status: NEW → ASSIGNED

All browsers have dropped supporting the legacy DOM mutation events and
there are some crash tests under testing/web-platform/tests/editing
because Gecko supported them even if the mutation is caused by the
builtin editor. I've already filed a spec issue for this [1].

This patch removes all tests which we added, however, there is only one
exception, that is insertAdjacentElement-with-DOMSubtreeModified.html.
Although it's not related to editing, but no body worked on moving it
into the proper directly (I filed an issue for this one year ago [2]).
Therefore, I leave it as-is but removing the DOMSubtreeModified event
listener because it seems that the listener triggers flushing pending
things. So, the other part may be still valuable for its author.

  1. https://github.com/web-platform-tests/wpt/issues/54682
  2. https://github.com/web-platform-tests/wpt/issues/48874
Pushed by masayuki@d-toybox.com: https://github.com/mozilla-firefox/firefox/commit/bb27cf413703 https://hg.mozilla.org/integration/autoland/rev/2b69d9067353 Get rid of editing tests which depend on the legacy DOM mutation events r=smaug,dom-core
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 146 Branch
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/55635 for changes under testing/web-platform/tests
Upstream PR merged by moz-wptsync-bot
QA Whiteboard: [qa-triage-done-c147/b146]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: