onwebkit{animation, transition}XX handlers missing from Document
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Mar 2020 14:11:32 +0000 (14:11 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 19 Mar 2020 14:11:32 +0000 (14:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=206170
<rdar://problem/58596373>

Patch by Antoine Quint <graouts@apple.com> on 2020-03-19
Reviewed by Youenn Fablet.

Source/WebCore:

We now specify the non-standard CSS Animations and CSS Transitions event handlers on DocumentAndElementEventHandlers.idl rather than Element.idl
such that they specified on both Element and Document.

* dom/DocumentAndElementEventHandlers.idl:
* dom/Element.idl:

LayoutTests:

* fast/dom/event-handler-attributes-expected.txt:
* fast/dom/event-handler-attributes.html:

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

LayoutTests/ChangeLog
LayoutTests/fast/dom/event-handler-attributes-expected.txt
LayoutTests/fast/dom/event-handler-attributes.html
Source/WebCore/ChangeLog
Source/WebCore/dom/DocumentAndElementEventHandlers.idl
Source/WebCore/dom/Element.idl

index 6374aae..adc60f6 100644 (file)
@@ -1,3 +1,14 @@
+2020-03-19  Antoine Quint  <graouts@apple.com>
+
+        onwebkit{animation, transition}XX handlers missing from Document
+        https://bugs.webkit.org/show_bug.cgi?id=206170
+        <rdar://problem/58596373>
+
+        Reviewed by Youenn Fablet.
+
+        * fast/dom/event-handler-attributes-expected.txt:
+        * fast/dom/event-handler-attributes.html:
+
 2020-03-19  Enrique Ocaña González  <eocanha@igalia.com>
 
         [GTK] media/track/track-automatic-subtitles.html is timing out
index b824e7f..bb4932c 100644 (file)
@@ -166,6 +166,10 @@ PASS testScriptAttribute(document, "timeupdate") is "document"
 PASS testScriptAttribute(document, "transitionend") is "document"
 PASS testScriptAttribute(document, "volumechange") is "document"
 PASS testScriptAttribute(document, "waiting") is "document"
+PASS testScriptAttribute(document, "webkitanimationend") is "document"
+PASS testScriptAttribute(document, "webkitanimationiteration") is "document"
+PASS testScriptAttribute(document, "webkitanimationstart") is "document"
+PASS testScriptAttribute(document, "webkittransitionend") is "document"
 PASS testScriptAttribute(document, "wheel") is "document"
 
 Event names we expect not to be able to set on the document
@@ -184,10 +188,6 @@ PASS testScriptAttribute(document, "pageshow") is "none"
 PASS testScriptAttribute(document, "popstate") is "none"
 PASS testScriptAttribute(document, "storage") is "none"
 PASS testScriptAttribute(document, "unload") is "none"
-PASS testScriptAttribute(document, "webkitanimationend") is "none"
-PASS testScriptAttribute(document, "webkitanimationiteration") is "none"
-PASS testScriptAttribute(document, "webkitanimationstart") is "none"
-PASS testScriptAttribute(document, "webkittransitionend") is "none"
 
 Event names we expect to be able to set on an element (tested on the <html> element)
 
index 1082e40..6410729 100644 (file)
@@ -72,7 +72,11 @@ const eventsInGlobalEventHandlers = [
     "volumechange",
     "waiting",
     "search",
-    "wheel"
+    "wheel",
+    "webkitanimationend",
+    "webkitanimationiteration",
+    "webkitanimationstart",
+    "webkittransitionend"
 ];
 
 const eventsInWindowEventHandlers = [
@@ -105,9 +109,6 @@ const elementAndWindowEvents = [
     "webkittransitionend"
 ];
 
-const uniqueWindowEvents = [
-];
-
 const uniqueDocumentEvents = [
     "selectionchange",
     "readystatechange"
@@ -246,7 +247,7 @@ function arrayDifference(a, b)
     });
 }
 
-const windowEvents = arrayUnion(eventsInGlobalEventHandlers, eventsInWindowEventHandlers, elementAndWindowEvents, uniqueWindowEvents);
+const windowEvents = arrayUnion(eventsInGlobalEventHandlers, eventsInWindowEventHandlers, elementAndWindowEvents);
 const documentEvents = arrayUnion(eventsInGlobalEventHandlers, elementAndDocumentEvents, uniqueDocumentEvents);
 const elementEvents = arrayUnion(eventsInGlobalEventHandlers, elementAndDocumentEvents, elementAndWindowEvents, elementBodyAndFrameSetEvents, uniqueElementEvents);
 
index e99dab7..d5bfd26 100644 (file)
@@ -1,3 +1,17 @@
+2020-03-19  Antoine Quint  <graouts@apple.com>
+
+        onwebkit{animation, transition}XX handlers missing from Document
+        https://bugs.webkit.org/show_bug.cgi?id=206170
+        <rdar://problem/58596373>
+
+        Reviewed by Youenn Fablet.
+
+        We now specify the non-standard CSS Animations and CSS Transitions event handlers on DocumentAndElementEventHandlers.idl rather than Element.idl
+        such that they specified on both Element and Document.
+
+        * dom/DocumentAndElementEventHandlers.idl:
+        * dom/Element.idl:
+
 2020-03-19  Carlos Garcia Campos  <cgarcia@igalia.com>
 
         [GTK][WPE] Media controls numeric position value is not automatically updated during playback
index 503fe3f..4f165ea 100644 (file)
@@ -36,4 +36,9 @@
     [NotEnumerable] attribute EventHandler onbeforecut;
     [NotEnumerable] attribute EventHandler onbeforeinput;
     [NotEnumerable] attribute EventHandler onbeforepaste;
+
+    [NotEnumerable, ImplementedAs=onwebkitAnimationEnd] attribute EventHandler onwebkitanimationend;
+    [NotEnumerable, ImplementedAs=onwebkitAnimationIteration] attribute EventHandler onwebkitanimationiteration;
+    [NotEnumerable, ImplementedAs=onwebkitAnimationStart] attribute EventHandler onwebkitanimationstart;
+    [NotEnumerable, ImplementedAs=onwebkitTransitionEnd] attribute EventHandler onwebkittransitionend;
 };
index 7368c32..9bb75d6 100644 (file)
     [NotEnumerable, Conditional=IOS_GESTURE_EVENTS] attribute EventHandler ongestureend;
     [NotEnumerable, Conditional=IOS_GESTURE_EVENTS] attribute EventHandler ongesturestart;
 
-    // Non standard.
-    [NotEnumerable, ImplementedAs=onwebkitAnimationEnd] attribute EventHandler onwebkitanimationend;
-    [NotEnumerable, ImplementedAs=onwebkitAnimationIteration] attribute EventHandler onwebkitanimationiteration;
-    [NotEnumerable, ImplementedAs=onwebkitAnimationStart] attribute EventHandler onwebkitanimationstart;
-    [NotEnumerable, ImplementedAs=onwebkitTransitionEnd] attribute EventHandler onwebkittransitionend;
-
     // FIXME: These event handlers should only be on Document (https://fullscreen.spec.whatwg.org/#api).
     [NotEnumerable, Conditional=FULLSCREEN_API, EnabledBySetting=FullScreen, DisabledByQuirk=shouldDisableElementFullscreen] attribute EventHandler onwebkitfullscreenchange;
     [NotEnumerable, Conditional=FULLSCREEN_API, EnabledBySetting=FullScreen, DisabledByQuirk=shouldDisableElementFullscreen] attribute EventHandler onwebkitfullscreenerror;