Title: [257978] trunk
Revision
257978
Author
[email protected]
Date
2020-03-06 06:22:08 -0800 (Fri, 06 Mar 2020)

Log Message

Remove the experimental flag for Pointer Events
https://bugs.webkit.org/show_bug.cgi?id=208655
<rdar://problem/60090545>

Reviewed by Dean Jackson.

Source/WebCore:

* dom/Element.cpp:
(WebCore::dispatchPointerEventIfNeeded):
(WebCore::Element::removedFromAncestor):
* dom/Element.idl:
* dom/NavigatorMaxTouchPoints.idl:
* dom/PointerEvent.idl:
* page/RuntimeEnabledFeatures.h:
(WebCore::RuntimeEnabledFeatures::setPointerEventsEnabled): Deleted.
(WebCore::RuntimeEnabledFeatures::pointerEventsEnabled const): Deleted.
* style/StyleTreeResolver.cpp:
(WebCore::Style::TreeResolver::resolveElement):

Source/WebKit:

* Shared/WebPreferences.yaml:

Source/WebKitLegacy/mac:

* WebView/WebPreferenceKeysPrivate.h:
* WebView/WebPreferences.mm:
(+[WebPreferences initialize]):
(-[WebPreferences syntheticEditingCommandsEnabled]):
(-[WebPreferences setSyntheticEditingCommandsEnabled:]):
(-[WebPreferences pointerEventsEnabled]): Deleted.
(-[WebPreferences setPointerEventsEnabled:]): Deleted.
* WebView/WebPreferencesPrivate.h:
* WebView/WebView.mm:

LayoutTests:

* fast/events/pointer/ios/tap-gives-pointerdown-pointerup.html:
* pointerevents/disabled-expected.html: Removed.
* pointerevents/disabled.html: Removed.

Modified Paths

Removed Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (257977 => 257978)


--- trunk/LayoutTests/ChangeLog	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/LayoutTests/ChangeLog	2020-03-06 14:22:08 UTC (rev 257978)
@@ -1,3 +1,15 @@
+2020-03-06  Antoine Quint  <[email protected]>
+
+        Remove the experimental flag for Pointer Events
+        https://bugs.webkit.org/show_bug.cgi?id=208655
+        <rdar://problem/60090545>
+
+        Reviewed by Dean Jackson.
+
+        * fast/events/pointer/ios/tap-gives-pointerdown-pointerup.html:
+        * pointerevents/disabled-expected.html: Removed.
+        * pointerevents/disabled.html: Removed.
+
 2020-03-06  Enrique Ocaña González  <[email protected]>
 
         [GStreamer] Streaming aac/mp3 audio doesn't always work

Modified: trunk/LayoutTests/fast/events/pointer/ios/tap-gives-pointerdown-pointerup.html (257977 => 257978)


--- trunk/LayoutTests/fast/events/pointer/ios/tap-gives-pointerdown-pointerup.html	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/LayoutTests/fast/events/pointer/ios/tap-gives-pointerdown-pointerup.html	2020-03-06 14:22:08 UTC (rev 257978)
@@ -1,4 +1,4 @@
-<!DOCTYPE html><!-- webkit-test-runner [ experimental:PointerEventsEnabled=true ] -->
+<!DOCTYPE html>
 <html>
 <head>
     <script src=""

Deleted: trunk/LayoutTests/pointerevents/disabled-expected.html (257977 => 257978)


--- trunk/LayoutTests/pointerevents/disabled-expected.html	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/LayoutTests/pointerevents/disabled-expected.html	2020-03-06 14:22:08 UTC (rev 257978)
@@ -1,3 +0,0 @@
-<!DOCTYPE html>
-<p>The PointerEvent interface shouldn't be visible when the feature is disabled.</p>
-<p>window.PointerEvent == <span>undefined</span></p>

Deleted: trunk/LayoutTests/pointerevents/disabled.html (257977 => 257978)


--- trunk/LayoutTests/pointerevents/disabled.html	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/LayoutTests/pointerevents/disabled.html	2020-03-06 14:22:08 UTC (rev 257978)
@@ -1,9 +0,0 @@
-<!DOCTYPE html><!-- webkit-test-runner [ experimental:PointerEventsEnabled=false ] -->
-<p>The PointerEvent interface shouldn't be visible when the feature is disabled.</p>
-<p>window.PointerEvent == <span></span></p>
-<script>
-window.addEventListener("load", () => {
-    const output = document.querySelector("span");
-    output.textContent = `${window.PointerEvent}`;
-}, false);
-</script>

Modified: trunk/Source/WebCore/ChangeLog (257977 => 257978)


--- trunk/Source/WebCore/ChangeLog	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebCore/ChangeLog	2020-03-06 14:22:08 UTC (rev 257978)
@@ -1,3 +1,23 @@
+2020-03-06  Antoine Quint  <[email protected]>
+
+        Remove the experimental flag for Pointer Events
+        https://bugs.webkit.org/show_bug.cgi?id=208655
+        <rdar://problem/60090545>
+
+        Reviewed by Dean Jackson.
+
+        * dom/Element.cpp:
+        (WebCore::dispatchPointerEventIfNeeded):
+        (WebCore::Element::removedFromAncestor):
+        * dom/Element.idl:
+        * dom/NavigatorMaxTouchPoints.idl:
+        * dom/PointerEvent.idl:
+        * page/RuntimeEnabledFeatures.h:
+        (WebCore::RuntimeEnabledFeatures::setPointerEventsEnabled): Deleted.
+        (WebCore::RuntimeEnabledFeatures::pointerEventsEnabled const): Deleted.
+        * style/StyleTreeResolver.cpp:
+        (WebCore::Style::TreeResolver::resolveElement):
+
 2020-03-06  Enrique Ocaña González  <[email protected]>
 
         [GStreamer] Streaming aac/mp3 audio doesn't always work

Modified: trunk/Source/WebCore/dom/Element.cpp (257977 => 257978)


--- trunk/Source/WebCore/dom/Element.cpp	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebCore/dom/Element.cpp	2020-03-06 14:22:08 UTC (rev 257978)
@@ -320,27 +320,25 @@
 static ShouldIgnoreMouseEvent dispatchPointerEventIfNeeded(Element& element, const MouseEvent& mouseEvent, const PlatformMouseEvent& platformEvent, bool& didNotSwallowEvent)
 {
 #if ENABLE(POINTER_EVENTS)
-    if (RuntimeEnabledFeatures::sharedFeatures().pointerEventsEnabled()) {
-        if (auto* page = element.document().page()) {
-            auto& pointerCaptureController = page->pointerCaptureController();
+    if (auto* page = element.document().page()) {
+        auto& pointerCaptureController = page->pointerCaptureController();
 #if ENABLE(TOUCH_EVENTS)
-            if (platformEvent.pointerId() != mousePointerID && mouseEvent.type() != eventNames().clickEvent && pointerCaptureController.preventsCompatibilityMouseEventsForIdentifier(platformEvent.pointerId()))
-                return ShouldIgnoreMouseEvent::Yes;
+        if (platformEvent.pointerId() != mousePointerID && mouseEvent.type() != eventNames().clickEvent && pointerCaptureController.preventsCompatibilityMouseEventsForIdentifier(platformEvent.pointerId()))
+            return ShouldIgnoreMouseEvent::Yes;
 #else
-            UNUSED_PARAM(platformEvent);
+        UNUSED_PARAM(platformEvent);
 #endif
-            if (platformEvent.syntheticClickType() != NoTap)
-                return ShouldIgnoreMouseEvent::No;
+        if (platformEvent.syntheticClickType() != NoTap)
+            return ShouldIgnoreMouseEvent::No;
 
-            if (auto pointerEvent = pointerCaptureController.pointerEventForMouseEvent(mouseEvent)) {
-                pointerCaptureController.dispatchEvent(*pointerEvent, &element);
-                if (isCompatibilityMouseEvent(mouseEvent) && pointerCaptureController.preventsCompatibilityMouseEventsForIdentifier(pointerEvent->pointerId()))
+        if (auto pointerEvent = pointerCaptureController.pointerEventForMouseEvent(mouseEvent)) {
+            pointerCaptureController.dispatchEvent(*pointerEvent, &element);
+            if (isCompatibilityMouseEvent(mouseEvent) && pointerCaptureController.preventsCompatibilityMouseEventsForIdentifier(pointerEvent->pointerId()))
+                return ShouldIgnoreMouseEvent::Yes;
+            if (pointerEvent->defaultPrevented() || pointerEvent->defaultHandled()) {
+                didNotSwallowEvent = false;
+                if (pointerEvent->type() == eventNames().pointerdownEvent)
                     return ShouldIgnoreMouseEvent::Yes;
-                if (pointerEvent->defaultPrevented() || pointerEvent->defaultHandled()) {
-                    didNotSwallowEvent = false;
-                    if (pointerEvent->type() == eventNames().pointerdownEvent)
-                        return ShouldIgnoreMouseEvent::Yes;
-                }
             }
         }
     }
@@ -2200,7 +2198,7 @@
         document().page()->pointerLockController().elementRemoved(*this);
 #endif
 #if ENABLE(POINTER_EVENTS)
-    if (document().page() && RuntimeEnabledFeatures::sharedFeatures().pointerEventsEnabled())
+    if (document().page())
         document().page()->pointerCaptureController().elementWasRemoved(*this);
 #endif
 

Modified: trunk/Source/WebCore/dom/Element.idl (257977 => 257978)


--- trunk/Source/WebCore/dom/Element.idl	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebCore/dom/Element.idl	2020-03-06 14:22:08 UTC (rev 257978)
@@ -100,9 +100,9 @@
     [Conditional=FULLSCREEN_API, EnabledBySetting=FullScreen, DisabledByQuirk=shouldDisableElementFullscreen] void webkitRequestFullscreen(); // Prefixed W3C version.
 
     // Extensions from Pointer Events API (https://w3c.github.io/pointerevents/#extensions-to-the-element-interface).
-    [Conditional=POINTER_EVENTS, EnabledAtRuntime=PointerEvents, MayThrowException] void setPointerCapture(long pointerId);
-    [Conditional=POINTER_EVENTS, EnabledAtRuntime=PointerEvents, MayThrowException] void releasePointerCapture(long pointerId);
-    [Conditional=POINTER_EVENTS, EnabledAtRuntime=PointerEvents] boolean hasPointerCapture(long pointerId);
+    [Conditional=POINTER_EVENTS, MayThrowException] void setPointerCapture(long pointerId);
+    [Conditional=POINTER_EVENTS, MayThrowException] void releasePointerCapture(long pointerId);
+    [Conditional=POINTER_EVENTS] boolean hasPointerCapture(long pointerId);
 
     // Extensions from Pointer Lock API (https://w3c.github.io/pointerlock/#extensions-to-the-element-interface).
     [Conditional=POINTER_LOCK] void requestPointerLock();

Modified: trunk/Source/WebCore/dom/NavigatorMaxTouchPoints.idl (257977 => 257978)


--- trunk/Source/WebCore/dom/NavigatorMaxTouchPoints.idl	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebCore/dom/NavigatorMaxTouchPoints.idl	2020-03-06 14:22:08 UTC (rev 257978)
@@ -28,8 +28,7 @@
 
 [
     NoInterfaceObject,
-    Conditional=POINTER_EVENTS,
-    EnabledAtRuntime=PointerEvents
+    Conditional=POINTER_EVENTS
 ] interface MaxTouchPoints {
     readonly attribute long maxTouchPoints;
 };

Modified: trunk/Source/WebCore/dom/PointerEvent.idl (257977 => 257978)


--- trunk/Source/WebCore/dom/PointerEvent.idl	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebCore/dom/PointerEvent.idl	2020-03-06 14:22:08 UTC (rev 257978)
@@ -38,7 +38,6 @@
 
 [
     Conditional=POINTER_EVENTS,
-    EnabledAtRuntime=PointerEvents,
     DisabledByQuirk=shouldDisablePointerEvents,
     Constructor(DOMString type, optional PointerEventInit eventInitDict),
     Exposed=Window

Modified: trunk/Source/WebCore/page/RuntimeEnabledFeatures.h (257977 => 257978)


--- trunk/Source/WebCore/page/RuntimeEnabledFeatures.h	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebCore/page/RuntimeEnabledFeatures.h	2020-03-06 14:22:08 UTC (rev 257978)
@@ -174,9 +174,6 @@
     void setCSSCustomPropertiesAndValuesEnabled(bool isEnabled) { m_CSSCustomPropertiesAndValuesEnabled = isEnabled; }
     bool cssCustomPropertiesAndValuesEnabled() const { return m_CSSCustomPropertiesAndValuesEnabled; }
 
-    void setPointerEventsEnabled(bool isEnabled) { m_pointerEventsEnabled = isEnabled; }
-    bool pointerEventsEnabled() const { return m_pointerEventsEnabled; }
-
     void setSyntheticEditingCommandsEnabled(bool isEnabled) { m_syntheticEditingCommandsEnabled = isEnabled; }
     bool syntheticEditingCommandsEnabled() const { return m_syntheticEditingCommandsEnabled; }
 
@@ -452,7 +449,6 @@
     bool m_attrStyleEnabled { false };
     bool m_webAPIStatisticsEnabled { false };
     bool m_CSSCustomPropertiesAndValuesEnabled { false };
-    bool m_pointerEventsEnabled { true };
     bool m_syntheticEditingCommandsEnabled { true };
     bool m_dialogElementEnabled { false };
     bool m_webSQLEnabled { false };

Modified: trunk/Source/WebCore/style/StyleTreeResolver.cpp (257977 => 257978)


--- trunk/Source/WebCore/style/StyleTreeResolver.cpp	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebCore/style/StyleTreeResolver.cpp	2020-03-06 14:22:08 UTC (rev 257978)
@@ -246,7 +246,7 @@
 
 #if ENABLE(POINTER_EVENTS) && PLATFORM(IOS_FAMILY)
     // FIXME: Track this exactly.
-    if (update.style->touchActions() != TouchAction::Auto && !m_document.quirks().shouldDisablePointerEventsQuirk() && RuntimeEnabledFeatures::sharedFeatures().pointerEventsEnabled())
+    if (update.style->touchActions() != TouchAction::Auto && !m_document.quirks().shouldDisablePointerEventsQuirk())
         m_document.setMayHaveElementsWithNonAutoTouchAction();
 #endif
 

Modified: trunk/Source/WebKit/ChangeLog (257977 => 257978)


--- trunk/Source/WebKit/ChangeLog	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebKit/ChangeLog	2020-03-06 14:22:08 UTC (rev 257978)
@@ -1,3 +1,13 @@
+2020-03-06  Antoine Quint  <[email protected]>
+
+        Remove the experimental flag for Pointer Events
+        https://bugs.webkit.org/show_bug.cgi?id=208655
+        <rdar://problem/60090545>
+
+        Reviewed by Dean Jackson.
+
+        * Shared/WebPreferences.yaml:
+
 2020-03-05  Brent Fulgham  <[email protected]>
 
         [iOS] Remove reporting for some well-understood framebuffer routines

Modified: trunk/Source/WebKit/Shared/WebPreferences.yaml (257977 => 257978)


--- trunk/Source/WebKit/Shared/WebPreferences.yaml	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebKit/Shared/WebPreferences.yaml	2020-03-06 14:22:08 UTC (rev 257978)
@@ -1310,14 +1310,6 @@
   humanReadableDescription: "Enable Visual Viewport API"
   category: experimental
 
-PointerEventsEnabled:
-  type: bool
-  defaultValue: true
-  humanReadableName: "Pointer Events"
-  humanReadableDescription: "Enable Pointer Events"
-  webcoreBinding: RuntimeEnabledFeatures
-  category: experimental
-
 SyntheticEditingCommandsEnabled:
   type: bool
   defaultValue: true

Modified: trunk/Source/WebKitLegacy/mac/ChangeLog (257977 => 257978)


--- trunk/Source/WebKitLegacy/mac/ChangeLog	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebKitLegacy/mac/ChangeLog	2020-03-06 14:22:08 UTC (rev 257978)
@@ -1,3 +1,21 @@
+2020-03-06  Antoine Quint  <[email protected]>
+
+        Remove the experimental flag for Pointer Events
+        https://bugs.webkit.org/show_bug.cgi?id=208655
+        <rdar://problem/60090545>
+
+        Reviewed by Dean Jackson.
+
+        * WebView/WebPreferenceKeysPrivate.h:
+        * WebView/WebPreferences.mm:
+        (+[WebPreferences initialize]):
+        (-[WebPreferences syntheticEditingCommandsEnabled]):
+        (-[WebPreferences setSyntheticEditingCommandsEnabled:]):
+        (-[WebPreferences pointerEventsEnabled]): Deleted.
+        (-[WebPreferences setPointerEventsEnabled:]): Deleted.
+        * WebView/WebPreferencesPrivate.h:
+        * WebView/WebView.mm:
+
 2020-03-05  youenn fablet  <[email protected]>
 
         Export NowPlaying commands to GPUProcess when media playing in GPUProcess is enabled

Modified: trunk/Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h (257977 => 257978)


--- trunk/Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h	2020-03-06 14:22:08 UTC (rev 257978)
@@ -55,7 +55,6 @@
 #define WebKitWebAnimationsEnabledPreferenceKey @"WebKitWebAnimationsEnabled"
 #define WebKitWebAnimationsCompositeOperationsEnabledPreferenceKey @"WebKitWebAnimationsCompositeOperationsEnabled"
 #define WebKitWebAnimationsMutableTimelinesEnabledPreferenceKey @"WebKitWebAnimationsMutableTimelinesEnabled"
-#define WebKitPointerEventsEnabledPreferenceKey @"WebKitPointerEventsEnabled"
 #define WebKitSyntheticEditingCommandsEnabledPreferenceKey @"WebKitSyntheticEditingCommandsEnabled"
 #define WebKitWebSecurityEnabledPreferenceKey @"WebKitWebSecurityEnabled"
 #define WebKitAllowUniversalAccessFromFileURLsPreferenceKey @"WebKitAllowUniversalAccessFromFileURLs"

Modified: trunk/Source/WebKitLegacy/mac/WebView/WebPreferences.mm (257977 => 257978)


--- trunk/Source/WebKitLegacy/mac/WebView/WebPreferences.mm	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebKitLegacy/mac/WebView/WebPreferences.mm	2020-03-06 14:22:08 UTC (rev 257978)
@@ -657,7 +657,6 @@
 #endif
         @NO, WebKitDirectoryUploadEnabledPreferenceKey,
         @YES, WebKitWebAnimationsEnabledPreferenceKey,
-        @YES, WebKitPointerEventsEnabledPreferenceKey,
         @YES, WebKitSyntheticEditingCommandsEnabledPreferenceKey,
 
 #if PLATFORM(IOS_FAMILY)
@@ -3238,24 +3237,14 @@
     [self _setBoolValue:flag forKey:WebKitWebAnimationsMutableTimelinesEnabledPreferenceKey];
 }
 
-- (BOOL)pointerEventsEnabled
-{
-    return [self _boolValueForKey:WebKitPointerEventsEnabledPreferenceKey];
-}
-
-- (void)setPointerEventsEnabled:(BOOL)flag
-{
-    [self _setBoolValue:flag forKey:WebKitPointerEventsEnabledPreferenceKey];
-}
-
 - (BOOL)syntheticEditingCommandsEnabled
 {
-    return [self _boolValueForKey:WebKitPointerEventsEnabledPreferenceKey];
+    return [self _boolValueForKey:WebKitSyntheticEditingCommandsEnabledPreferenceKey];
 }
 
 - (void)setSyntheticEditingCommandsEnabled:(BOOL)flag
 {
-    [self _setBoolValue:flag forKey:WebKitPointerEventsEnabledPreferenceKey];
+    [self _setBoolValue:flag forKey:WebKitSyntheticEditingCommandsEnabledPreferenceKey];
 }
 
 - (BOOL)fetchAPIKeepAliveEnabled

Modified: trunk/Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h (257977 => 257978)


--- trunk/Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h	2020-03-06 14:22:08 UTC (rev 257978)
@@ -586,9 +586,6 @@
 - (void)setWebAnimationsMutableTimelinesEnabled:(BOOL)flag;
 - (BOOL)webAnimationsMutableTimelinesEnabled;
 
-- (void)setPointerEventsEnabled:(BOOL)flag;
-- (BOOL)pointerEventsEnabled;
-
 - (void)setSyntheticEditingCommandsEnabled:(BOOL)flag;
 - (BOOL)syntheticEditingCommandsEnabled;
 

Modified: trunk/Source/WebKitLegacy/mac/WebView/WebView.mm (257977 => 257978)


--- trunk/Source/WebKitLegacy/mac/WebView/WebView.mm	2020-03-06 12:41:27 UTC (rev 257977)
+++ trunk/Source/WebKitLegacy/mac/WebView/WebView.mm	2020-03-06 14:22:08 UTC (rev 257978)
@@ -3186,10 +3186,6 @@
     RuntimeEnabledFeatures::sharedFeatures().setWebAnimationsCompositeOperationsEnabled([preferences webAnimationsCompositeOperationsEnabled]);
     RuntimeEnabledFeatures::sharedFeatures().setWebAnimationsMutableTimelinesEnabled([preferences webAnimationsMutableTimelinesEnabled]);
 
-#if ENABLE(POINTER_EVENTS)
-    RuntimeEnabledFeatures::sharedFeatures().setPointerEventsEnabled([preferences pointerEventsEnabled]);
-#endif
-
 #if ENABLE(INTERSECTION_OBSERVER)
     RuntimeEnabledFeatures::sharedFeatures().setIntersectionObserverEnabled(preferences.intersectionObserverEnabled);
 #endif
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to