Bug 769207 - part 14: Get rid of the pref to enable the legacy DOM mutation events and delete tests checking or depending on mutation events r=smaug,dom-core

Differential Revision: https://phabricator.services.mozilla.com/D263065

Author: masayuki-nakano

docshell/base/crashtests/432114-2.html

@@ -3,7 +3,6 @@ load 369126-1.html
 load 430124-1.html
 load 430628-1.html
 load 432114-1.html
-load 432114-2.html
 load 436900-1.html
 asserts(0-1) load 436900-2.html # bug 566159
 load 443655.html

docshell/base/crashtests/crashtests.list

@@ -8195,16 +8195,6 @@ void Document::SetScopeObject(nsIGlobalObject* aGlobal) {
     MOZ_ASSERT_IF(
         mNodeInfoManager->GetArenaAllocator(),
         mNodeInfoManager->GetArenaAllocator() == mDocGroup->ArenaAllocator());
-
-    // Update data document's mMutationEventsEnabled early on so that we can
-    // avoid extra IsURIInPrefList calls.
-    if (mLoadedAsData && window->GetExtantDoc() &&
-        window->GetExtantDoc() != this &&
-        window->GetExtantDoc()->NodePrincipal() == NodePrincipal() &&
-        mMutationEventsEnabled.isNothing()) {
-      mMutationEventsEnabled.emplace(
-          window->GetExtantDoc()->MutationEventsEnabled());
-    }
   }
 }
 
@@ -20744,17 +20734,6 @@ already_AddRefed<Document> Document::ParseHTML(GlobalObject& aGlobal,
   return doc.forget();
 }
 
-bool Document::MutationEventsEnabled() {
-  if (StaticPrefs::dom_mutation_events_enabled()) {
-    return true;
-  }
-  if (mMutationEventsEnabled.isNothing()) {
-    mMutationEventsEnabled.emplace(
-        NodePrincipal()->IsURIInPrefList("dom.mutation_events.forceEnable"));
-  }
-  return mMutationEventsEnabled.value();
-}
-
 void Document::GetAllInProcessDocuments(
     nsTArray<RefPtr<Document>>& aAllDocuments) {
   for (Document* doc : AllDocumentsList()) {

docshell/base/crashtests/file_432114-2.xhtml

@@ -3789,10 +3789,6 @@ class Document : public nsINode,
   void ClearClipboardCopyTriggered() { mClipboardCopyTriggered = false; }
   void SetClipboardCopyTriggered() { mClipboardCopyTriggered = true; }
 
-  // Even if mutation events are disabled by default,
-  // dom.mutation_events.forceEnable can be used to enable them per site.
-  bool MutationEventsEnabled();
-
   // This should be called when this document receives events which are likely
   // to be user interaction with the document, rather than the byproduct of
   // interaction with the browser (i.e. a keypress to scroll the view port,
@@ -5098,8 +5094,6 @@ class Document : public nsINode,
   // while a paste event is being handled in JS.
   bool mClipboardCopyTriggered : 1;
 
-  Maybe<bool> mMutationEventsEnabled;
-
   // The fingerprinting protections overrides for this document. The value will
   // override the default enabled fingerprinting protections for this document.
   // This will only get populated if these is one that comes from the local

dom/base/Document.cpp

@@ -17,7 +17,6 @@ load 327695-1.html
 load 329481-1.xhtml
 load 336381-1.xhtml
 load 336715-1.xhtml
-load 338391-1.xhtml
 load 338674-1.xhtml
 load 340733-1.html
 load 343730-1.xhtml

dom/base/Document.h

@@ -386,10 +386,6 @@ skip-if = [
   "http2",
 ]
 
-["test_bug339494.html"]
-
-["test_bug339494.xhtml"]
-
 ["test_bug343596.html"]
 
 ["test_bug345339.html"]
@@ -419,18 +415,10 @@ support-files = ["file_bug357450.js"]
 
 ["test_bug358660.html"]
 
-["test_bug362391.xhtml"]
-
-["test_bug364092.xhtml"]
-
-["test_bug364413.xhtml"]
-
 ["test_bug366944.html"]
 
 ["test_bug366946.html"]
 
-["test_bug367164.html"]
-
 ["test_bug368972.html"]
 
 ["test_bug371576-2.html"]
@@ -500,8 +488,6 @@ support-files = ["test_bug402150.html^headers^"]
 
 ["test_bug403868.xml"]
 
-["test_bug405182.html"]
-
 ["test_bug409380.html"]
 
 ["test_bug410229.html"]
@@ -732,8 +718,6 @@ support-files = ["file_bug518104.js"]
 
 ["test_bug562169-2.html"]
 
-["test_bug562652.html"]
-
 ["test_bug564047.html"]
 
 ["test_bug564863.xhtml"]
@@ -914,8 +898,6 @@ skip-if = [
 
 ["test_bug719533.html"]
 
-["test_bug726364.html"]
-
 ["test_bug737087.html"]
 
 ["test_bug737565.html"]
@@ -1151,8 +1133,6 @@ support-files = ["bug1739957.sjs"]
 
 ["test_bug1784187.html"]
 
-["test_bug1799354.html"]
-
 ["test_caretPositionFromPoint.html"]
 
 ["test_change_policy.html"]

dom/base/crashtests/338391-1.xhtml

@@ -41,42 +41,11 @@
   "Should be UPPERCASE AGAIN!");
   is(document.documentElement.getAttributeNS("", "AA"), "UPPERCASE", "Should be UPPERCASE!");
 
-  var removedNodeAccordingToEvent;
-
-  function mutationHandler(aEvent) {
-    removedNodeAccordingToEvent = aEvent.relatedNode;
-  }
-
-  var test1 = document.createElement("div");
-  test1.setAttribute("x", "y");
-  removedNodeAccordingToEvent = null;
-
-  function testremoveNamedItemNS() {
-    test1.addEventListener("DOMAttrModified", mutationHandler, true);
-    var removedNodeAccordingToRemoveNamedItemNS = test1.attributes.removeNamedItemNS(null, "x");
-    test1.removeEventListener("DOMAttrModified", mutationHandler, true);
-    is(removedNodeAccordingToEvent, removedNodeAccordingToRemoveNamedItemNS, "Node removed according to event is not same as node removed by removeNamedItemNS.");
-  }
-
-  testremoveNamedItemNS();
-
-  var test2 = document.createElement("div");
-  test2.setAttribute("x", "y");
-  removedNodeAccordingToEvent = null;
-
-  function testremoveNamedItem() {
-    test2.addEventListener("DOMAttrModified", mutationHandler, true);
-    var removedNodeAccordingToRemoveNamedItem = test2.attributes.removeNamedItem("x");
-    test2.removeEventListener("DOMAttrModified", mutationHandler, true);
-    is(removedNodeAccordingToEvent, removedNodeAccordingToRemoveNamedItem, "Node removed according to event is not same as node removed by removeNamedItem.");
-  }
-
-  testremoveNamedItem();
   SimpleTest.finish();
 }
 
 SimpleTest.waitForExplicitFinish();
-SpecialPowers.pushPrefEnv({"set": [["dom.mutation_events.enabled", true]]}, test);
+SimpleTest.executeSoon(test);
 </script>
 </body>
 </html>