Title: [250880] branches/safari-608.3.10.0-branch/Source/WebKit
- Revision
- 250880
- Author
- alanc...@apple.com
- Date
- 2019-10-08 17:52:24 -0700 (Tue, 08 Oct 2019)
Log Message
Revert "Cherry-pick r250755. rdar://problem/56061130"
This reverts commit r250829.
Modified Paths
Diff
Modified: branches/safari-608.3.10.0-branch/Source/WebKit/ChangeLog (250879 => 250880)
--- branches/safari-608.3.10.0-branch/Source/WebKit/ChangeLog 2019-10-09 00:52:18 UTC (rev 250879)
+++ branches/safari-608.3.10.0-branch/Source/WebKit/ChangeLog 2019-10-09 00:52:24 UTC (rev 250880)
@@ -1,51 +1,5 @@
2019-10-08 Alan Coon <alanc...@apple.com>
- Cherry-pick r250755. rdar://problem/56061130
-
- Use a better name than allowFastClicksEverywhere
- https://bugs.webkit.org/show_bug.cgi?id=202607
- <rdar://problem/55997133>
-
- Reviewed by Tim Horton.
-
- This preference name is quite confusing. Change it to
- PreferFasterClickOverDoubleTap.
-
- * Shared/WebPreferences.yaml:
- * Shared/WebPreferencesDefaultValues.h:
- * UIProcess/WebPageProxy.h:
- (WebKit::WebPageProxy::preferFasterClickOverDoubleTap const):
- (WebKit::WebPageProxy::allowsFastClicksEverywhere const): Deleted.
- * UIProcess/ios/WKContentViewInteraction.mm:
- (-[WKContentView _handleSmartMagnificationInformationForPotentialTap:renderRect:fitEntireRect:viewportMinimumScale:viewportMaximumScale:]):
- * UIProcess/ios/WebPageProxyIOS.mm:
- (WebKit::WebPageProxy::effectiveContentModeAfterAdjustingPolicies):
-
- git-svn-id: https://svn.webkit.org/repository/webkit/trunk@250755 268f45cc-cd09-0410-ab3c-d52691b4dbfc
-
- 2019-10-04 Dean Jackson <d...@apple.com>
-
- Use a better name than allowFastClicksEverywhere
- https://bugs.webkit.org/show_bug.cgi?id=202607
- <rdar://problem/55997133>
-
- Reviewed by Tim Horton.
-
- This preference name is quite confusing. Change it to
- PreferFasterClickOverDoubleTap.
-
- * Shared/WebPreferences.yaml:
- * Shared/WebPreferencesDefaultValues.h:
- * UIProcess/WebPageProxy.h:
- (WebKit::WebPageProxy::preferFasterClickOverDoubleTap const):
- (WebKit::WebPageProxy::allowsFastClicksEverywhere const): Deleted.
- * UIProcess/ios/WKContentViewInteraction.mm:
- (-[WKContentView _handleSmartMagnificationInformationForPotentialTap:renderRect:fitEntireRect:viewportMinimumScale:viewportMaximumScale:]):
- * UIProcess/ios/WebPageProxyIOS.mm:
- (WebKit::WebPageProxy::effectiveContentModeAfterAdjustingPolicies):
-
-2019-10-08 Alan Coon <alanc...@apple.com>
-
Cherry-pick r250438. rdar://problem/55984974
Unreviewed build-fix: wrap the AssertionServicesSPI.h in an IOS_FAMILY check.
Modified: branches/safari-608.3.10.0-branch/Source/WebKit/Shared/WebPreferences.yaml (250879 => 250880)
--- branches/safari-608.3.10.0-branch/Source/WebKit/Shared/WebPreferences.yaml 2019-10-09 00:52:18 UTC (rev 250879)
+++ branches/safari-608.3.10.0-branch/Source/WebKit/Shared/WebPreferences.yaml 2019-10-09 00:52:24 UTC (rev 250880)
@@ -1534,17 +1534,17 @@
type: bool
defaultValue: true
condition: PLATFORM(IOS_FAMILY)
- humanReadableName: "Fast clicks"
+ humanReadableName: "Faster clicks"
humanReadableDescription: "Support faster clicks on zoomable pages"
webcoreBinding: none
category: internal
-PreferFasterClickOverDoubleTap:
+FastClicksEverywhere:
type: bool
- defaultValue: DEFAULT_PREFER_FASTER_CLICK_OVER_DOUBLE_TAP
+ defaultValue: DEFAULT_FAST_CLICKS_EVERYWHERE
condition: PLATFORM(IOS_FAMILY)
- humanReadableName: "Use fast clicks before double tap"
- humanReadableDescription: "Prefer a faster click over a double tap"
+ humanReadableName: "Fast clicks everywhere"
+ humanReadableDescription: "Force fast clicks on all pages"
webcoreBinding: none
category: internal
Modified: branches/safari-608.3.10.0-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.h (250879 => 250880)
--- branches/safari-608.3.10.0-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.h 2019-10-09 00:52:18 UTC (rev 250879)
+++ branches/safari-608.3.10.0-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.h 2019-10-09 00:52:24 UTC (rev 250880)
@@ -256,11 +256,13 @@
#define DEFAULT_DOM_PASTE_ACCESS_REQUESTS_ENABLED false
#endif
-#if PLATFORM(IOS_FAMILY) && !PLATFORM(WATCHOS)
-#define DEFAULT_PREFER_FASTER_CLICK_OVER_DOUBLE_TAP true
+#if PLATFORM(IOS_FAMILY)
+#if PLATFORM(WATCHOS)
+#define DEFAULT_FAST_CLICKS_EVERYWHERE false
#else
-#define DEFAULT_PREFER_FASTER_CLICK_OVER_DOUBLE_TAP false
+#define DEFAULT_FAST_CLICKS_EVERYWHERE true
#endif
+#endif
#if ENABLE(APPLE_PAY_REMOTE_UI)
#define DEFAULT_APPLE_PAY_ENABLED true
Modified: branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/WebPageProxy.h (250879 => 250880)
--- branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/WebPageProxy.h 2019-10-09 00:52:18 UTC (rev 250879)
+++ branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/WebPageProxy.h 2019-10-09 00:52:24 UTC (rev 250880)
@@ -516,7 +516,7 @@
bool hasCommittedAnyProvisionalLoads() const { return m_hasCommittedAnyProvisionalLoads; }
- bool preferFasterClickOverDoubleTap() const { return m_preferFasterClickOverDoubleTap; }
+ bool allowsFastClicksEverywhere() const { return m_allowsFastClicksEverywhere; }
void setIsUsingHighPerformanceWebGL(bool value) { m_isUsingHighPerformanceWebGL = value; }
bool isUsingHighPerformanceWebGL() const { return m_isUsingHighPerformanceWebGL; }
@@ -2500,7 +2500,7 @@
bool m_isUsingHighPerformanceWebGL { false };
bool m_openedByDOM { false };
bool m_hasCommittedAnyProvisionalLoads { false };
- bool m_preferFasterClickOverDoubleTap { false };
+ bool m_allowsFastClicksEverywhere { false };
HashMap<String, Ref<WebURLSchemeHandler>> m_urlSchemeHandlersByScheme;
HashMap<uint64_t, Ref<WebURLSchemeHandler>> m_urlSchemeHandlersByIdentifier;
Modified: branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm (250879 => 250880)
--- branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm 2019-10-09 00:52:18 UTC (rev 250879)
+++ branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm 2019-10-09 00:52:24 UTC (rev 250880)
@@ -1728,8 +1728,8 @@
if (!_potentialTapInProgress)
return;
- if (_page->preferences().preferFasterClickOverDoubleTap() && _page->preferFasterClickOverDoubleTap()) {
- RELEASE_LOG(ViewGestures, "Potential tap found an element and fast taps are preferred. Trigger click. (%p)", self);
+ if (_page->preferences().fastClicksEverywhere() && _page->allowsFastClicksEverywhere()) {
+ RELEASE_LOG(ViewGestures, "Potential tap found an element and fast taps are forced on. Trigger click. (%p)", self);
[self _setDoubleTapGesturesEnabled:NO];
return;
}
Modified: branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm (250879 => 250880)
--- branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm 2019-10-09 00:52:18 UTC (rev 250879)
+++ branches/safari-608.3.10.0-branch/Source/WebKit/UIProcess/ios/WebPageProxyIOS.mm 2019-10-09 00:52:24 UTC (rev 250880)
@@ -1451,7 +1451,7 @@
break;
}
- m_preferFasterClickOverDoubleTap = false;
+ m_allowsFastClicksEverywhere = false;
if (!useDesktopBrowsingMode) {
policies.setAllowContentChangeObserverQuirk(true);
@@ -1476,7 +1476,7 @@
policies.setMediaSourcePolicy(WebsiteMediaSourcePolicy::Enable);
policies.setSimulatedMouseEventsDispatchPolicy(WebsiteSimulatedMouseEventsDispatchPolicy::Allow);
policies.setLegacyOverflowScrollingTouchPolicy(WebsiteLegacyOverflowScrollingTouchPolicy::Disable);
- m_preferFasterClickOverDoubleTap = true;
+ m_allowsFastClicksEverywhere = true;
}
return WebContentMode::Desktop;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes