Changeset 244665 in webkit

Timestamp:

Apr 25, 2019, 2:42:50 PM (7 years ago)

Author:

timothy@apple.com

Message:

Disable ContentChangeObserver on iOSMac.
​https://bugs.webkit.org/show_bug.cgi?id=197292
rdar://problem/49039957

Reviewed by Zalan Bujtas.

We don’t need to run any of ContentChangeObserver, because we have hover events on iOSMac.
Disabling it skips the synthetic mouse move events and speeds up clicks.

• page/SettingsBase.cpp:

(WebCore::SettingsBase::defaultContentChangeObserverEnabled): Return false for PLATFORM(IOSMAC).

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• page/SettingsBase.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2019-04-25  Timothy Hatcher  <timothy@apple.com>
+
+        Disable ContentChangeObserver on iOSMac.
+        https://bugs.webkit.org/show_bug.cgi?id=197292
+        rdar://problem/49039957
+
+        Reviewed by Zalan Bujtas.
+
+        We don’t need to run any of ContentChangeObserver, because we have hover events on iOSMac.
+        Disabling it skips the synthetic mouse move events and speeds up clicks.
+
+        * page/SettingsBase.cpp:
+        (WebCore::SettingsBase::defaultContentChangeObserverEnabled): Return false for PLATFORM(IOSMAC).
+
 2019-04-25  Timothy Hatcher  <timothy@apple.com>
 

trunk/Source/WebCore/page/SettingsBase.cpp

@@ -110 +110 @@
 bool SettingsBase::defaultContentChangeObserverEnabled()
 {
-#if PLATFORM(IOS_FAMILY)
+#if PLATFORM(IOS_FAMILY) && !PLATFORM(IOSMAC)
     return true;
 #else