It should be possible to re-initialize a MutationEvent after it's been dispatched
authorcdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 Aug 2016 14:49:20 +0000 (14:49 +0000)
committercdumez@apple.com <cdumez@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 10 Aug 2016 14:49:20 +0000 (14:49 +0000)
https://bugs.webkit.org/show_bug.cgi?id=160703

Reviewed by Sam Weinig.

Source/WebCore:

It should be possible to re-initialize a MutationEvent after it's been dispatched,
similarly to Event:
- https://www.w3.org/TR/2015/WD-uievents-20151215/#widl-MutationEvent-initMutationEvent

Test: fast/events/initMutationEvent-after-dispatch.html

* dom/MutationEvent.cpp:
(WebCore::MutationEvent::initMutationEvent):

LayoutTests:

Add layout test coverage.

* fast/events/initMutationEvent-after-dispatch-expected.txt: Added.
* fast/events/initMutationEvent-after-dispatch.html: Added.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@204331 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/events/initMutationEvent-after-dispatch-expected.txt [new file with mode: 0644]
LayoutTests/fast/events/initMutationEvent-after-dispatch.html [new file with mode: 0644]
Source/WebCore/ChangeLog
Source/WebCore/dom/MutationEvent.cpp

index 218b385..b1070fa 100644 (file)
@@ -1,3 +1,15 @@
+2016-08-10  Chris Dumez  <cdumez@apple.com>
+
+        It should be possible to re-initialize a MutationEvent after it's been dispatched
+        https://bugs.webkit.org/show_bug.cgi?id=160703
+
+        Reviewed by Sam Weinig.
+
+        Add layout test coverage.
+
+        * fast/events/initMutationEvent-after-dispatch-expected.txt: Added.
+        * fast/events/initMutationEvent-after-dispatch.html: Added.
+
 2016-08-09  Chris Dumez  <cdumez@apple.com>
 
         Optimization in Node::appendChild() is not spec-compliant
diff --git a/LayoutTests/fast/events/initMutationEvent-after-dispatch-expected.txt b/LayoutTests/fast/events/initMutationEvent-after-dispatch-expected.txt
new file mode 100644 (file)
index 0000000..f55e424
--- /dev/null
@@ -0,0 +1,55 @@
+Tests calling initMutationEvent() on a mutation event while it is being dispatched and after.
+
+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
+
+
+PASS testEvent.__proto__ is MutationEvent.prototype
+PASS testEvent.initMutationEvent("foo", true, false, document.body) did not throw exception.
+PASS testEvent.type is "foo"
+PASS testEvent.bubbles is true
+PASS testEvent.cancelable is false
+PASS testEvent.target is null
+PASS testEvent.currentTarget is null
+PASS testEvent.eventPhase is Event.NONE
+PASS testEvent.relatedNode is document.body
+
+* In event handler
+PASS event.type is "foo"
+PASS event.bubbles is true
+PASS event.cancelable is false
+PASS event.target is document.body
+PASS event.currentTarget is document.body
+PASS event.eventPhase is Event.AT_TARGET
+PASS event.relatedNode is document.body
+
+PASS event.initMutationEvent("bar", false, true, document.documentElement) did not throw exception.
+PASS event.type is "foo"
+PASS event.bubbles is true
+PASS event.cancelable is false
+PASS event.target is document.body
+PASS event.currentTarget is document.body
+PASS event.eventPhase is Event.AT_TARGET
+PASS event.relatedNode is document.body
+PASS document.body.dispatchEvent(testEvent) did not throw exception.
+
+* After event has been dispatched
+PASS testEvent.type is "foo"
+PASS testEvent.bubbles is true
+PASS testEvent.cancelable is false
+PASS testEvent.target is document.body
+PASS testEvent.currentTarget is null
+PASS testEvent.eventPhase is Event.NONE
+PASS testEvent.relatedNode is document.body
+
+PASS testEvent.initMutationEvent("bar", false, true, document.documentElement) did not throw exception.
+PASS testEvent.type is "bar"
+PASS testEvent.bubbles is false
+PASS testEvent.cancelable is true
+PASS testEvent.target is null
+PASS testEvent.currentTarget is null
+PASS testEvent.eventPhase is Event.NONE
+PASS testEvent.relatedNode is document.documentElement
+PASS successfullyParsed is true
+
+TEST COMPLETE
+
diff --git a/LayoutTests/fast/events/initMutationEvent-after-dispatch.html b/LayoutTests/fast/events/initMutationEvent-after-dispatch.html
new file mode 100644 (file)
index 0000000..d4d34c4
--- /dev/null
@@ -0,0 +1,69 @@
+<!DOCTYPE html>
+<html>
+<body>
+<script src="../../resources/js-test-pre.js"></script>
+<script>
+description("Tests calling initMutationEvent() on a mutation event while it is being dispatched and after.");
+
+function eventHandler(ev) {
+    event = ev;
+    debug("");
+    debug("* In event handler");
+    shouldBeEqualToString("event.type", "foo");
+    shouldBeTrue("event.bubbles");
+    shouldBeFalse("event.cancelable");
+    shouldBe("event.target", "document.body");
+    shouldBe("event.currentTarget", "document.body");
+    shouldBe("event.eventPhase", "Event.AT_TARGET");
+    shouldBe("event.relatedNode", "document.body");
+
+    debug("");
+    // It should not be possible to re-initialize an event being dispatched.
+    shouldNotThrow('event.initMutationEvent("bar", false, true, document.documentElement)');
+    shouldBeEqualToString("event.type", "foo");
+    shouldBeTrue("event.bubbles");
+    shouldBeFalse("event.cancelable");
+    shouldBe("event.target", "document.body");
+    shouldBe("event.currentTarget", "document.body");
+    shouldBe("event.eventPhase", "Event.AT_TARGET");
+    shouldBe("event.relatedNode", "document.body");
+}
+
+var testEvent = document.createEvent("mutationevent");
+shouldBe("testEvent.__proto__", "MutationEvent.prototype");
+shouldNotThrow('testEvent.initMutationEvent("foo", true, false, document.body)');
+shouldBeEqualToString("testEvent.type", "foo");
+shouldBeTrue("testEvent.bubbles");
+shouldBeFalse("testEvent.cancelable");
+shouldBeNull("testEvent.target");
+shouldBeNull("testEvent.currentTarget");
+shouldBe("testEvent.eventPhase", "Event.NONE");
+shouldBe("testEvent.relatedNode", "document.body");
+
+document.body.addEventListener("foo", eventHandler);
+shouldNotThrow('document.body.dispatchEvent(testEvent)');
+
+debug("");
+debug("* After event has been dispatched");
+shouldBeEqualToString("testEvent.type", "foo");
+shouldBeTrue("testEvent.bubbles");
+shouldBeFalse("testEvent.cancelable");
+shouldBe("testEvent.target", "document.body");
+shouldBeNull("testEvent.currentTarget");
+shouldBe("testEvent.eventPhase", "Event.NONE");
+shouldBe("testEvent.relatedNode", "document.body");
+
+// It should now be possible to re-initialize the event.
+debug("");
+shouldNotThrow('testEvent.initMutationEvent("bar", false, true, document.documentElement)');
+shouldBeEqualToString("testEvent.type", "bar");
+shouldBeFalse("testEvent.bubbles");
+shouldBeTrue("testEvent.cancelable");
+shouldBeNull("testEvent.target");
+shouldBeNull("testEvent.currentTarget");
+shouldBe("testEvent.eventPhase", "Event.NONE");
+shouldBe("testEvent.relatedNode", "document.documentElement");
+</script>
+<script src="../../resources/js-test-post.js"></script>
+</body>
+</html>
index a7661a9..2a2bac6 100644 (file)
@@ -1,3 +1,19 @@
+2016-08-10  Chris Dumez  <cdumez@apple.com>
+
+        It should be possible to re-initialize a MutationEvent after it's been dispatched
+        https://bugs.webkit.org/show_bug.cgi?id=160703
+
+        Reviewed by Sam Weinig.
+
+        It should be possible to re-initialize a MutationEvent after it's been dispatched,
+        similarly to Event:
+        - https://www.w3.org/TR/2015/WD-uievents-20151215/#widl-MutationEvent-initMutationEvent
+
+        Test: fast/events/initMutationEvent-after-dispatch.html
+
+        * dom/MutationEvent.cpp:
+        (WebCore::MutationEvent::initMutationEvent):
+
 2016-08-10  Yusuke Suzuki  <utatane.tea@gmail.com>
 
         [ES6] Add ModuleLoaderPrototype and move methods to it
index a61ef76..6382350 100644 (file)
@@ -51,7 +51,7 @@ void MutationEvent::initMutationEvent(const AtomicString& type, bool canBubble,
                                       const String& prevValue, const String& newValue,
                                       const String& attrName, unsigned short attrChange)
 {
-    if (dispatched())
+    if (isBeingDispatched())
         return;
 
     initEvent(type, canBubble, cancelable);