Title: [248574] branches/safari-608-branch/Source
- Revision
- 248574
- Author
- [email protected]
- Date
- 2019-08-12 16:42:34 -0700 (Mon, 12 Aug 2019)
Log Message
Cherry-pick r248471. rdar://problem/54130624
Disable CSSOM View Scrolling API for IMDb iOS app
https://bugs.webkit.org/show_bug.cgi?id=200586
<rdar://problem/53645833>
Patch by Alex Christensen <[email protected]> on 2019-08-09
Reviewed by Simon Fraser.
Source/WebCore:
They are calling scrollHeight on the HTML element and it is running new code introduced in r235806
Disable this new feature until they update their app to use the iOS13 SDK.
* platform/RuntimeApplicationChecks.h:
* platform/cocoa/RuntimeApplicationChecksCocoa.mm:
(WebCore::IOSApplication::isIMDb):
Source/WebKit:
Change the CSSOMViewScrollingAPIEnabled default value to be off for the IMDb app's WKWebViews.
I manually verified this is effective in those WKWebViews but no other WKWebViews and that it fixes the radar.
* Shared/WebPreferences.yaml:
* Shared/WebPreferencesDefaultValues.cpp:
(WebKit::defaultCSSOMViewScrollingAPIEnabled):
* Shared/WebPreferencesDefaultValues.h:
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248471 268f45cc-cd09-0410-ab3c-d52691b4dbfc
Modified Paths
Diff
Modified: branches/safari-608-branch/Source/WebCore/ChangeLog (248573 => 248574)
--- branches/safari-608-branch/Source/WebCore/ChangeLog 2019-08-12 23:42:30 UTC (rev 248573)
+++ branches/safari-608-branch/Source/WebCore/ChangeLog 2019-08-12 23:42:34 UTC (rev 248574)
@@ -1,5 +1,52 @@
2019-08-12 Alan Coon <[email protected]>
+ Cherry-pick r248471. rdar://problem/54130624
+
+ Disable CSSOM View Scrolling API for IMDb iOS app
+ https://bugs.webkit.org/show_bug.cgi?id=200586
+ <rdar://problem/53645833>
+
+ Patch by Alex Christensen <[email protected]> on 2019-08-09
+ Reviewed by Simon Fraser.
+
+ Source/WebCore:
+
+ They are calling scrollHeight on the HTML element and it is running new code introduced in r235806
+ Disable this new feature until they update their app to use the iOS13 SDK.
+
+ * platform/RuntimeApplicationChecks.h:
+ * platform/cocoa/RuntimeApplicationChecksCocoa.mm:
+ (WebCore::IOSApplication::isIMDb):
+
+ Source/WebKit:
+
+ Change the CSSOMViewScrollingAPIEnabled default value to be off for the IMDb app's WKWebViews.
+ I manually verified this is effective in those WKWebViews but no other WKWebViews and that it fixes the radar.
+
+ * Shared/WebPreferences.yaml:
+ * Shared/WebPreferencesDefaultValues.cpp:
+ (WebKit::defaultCSSOMViewScrollingAPIEnabled):
+ * Shared/WebPreferencesDefaultValues.h:
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248471 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2019-08-09 Alex Christensen <[email protected]>
+
+ Disable CSSOM View Scrolling API for IMDb iOS app
+ https://bugs.webkit.org/show_bug.cgi?id=200586
+ <rdar://problem/53645833>
+
+ Reviewed by Simon Fraser.
+
+ They are calling scrollHeight on the HTML element and it is running new code introduced in r235806
+ Disable this new feature until they update their app to use the iOS13 SDK.
+
+ * platform/RuntimeApplicationChecks.h:
+ * platform/cocoa/RuntimeApplicationChecksCocoa.mm:
+ (WebCore::IOSApplication::isIMDb):
+
+2019-08-12 Alan Coon <[email protected]>
+
Cherry-pick r248463. rdar://problem/54139782
REGRESSION (iOS 13): united.com web forms do not respond to taps
Modified: branches/safari-608-branch/Source/WebCore/platform/RuntimeApplicationChecks.h (248573 => 248574)
--- branches/safari-608-branch/Source/WebCore/platform/RuntimeApplicationChecks.h 2019-08-12 23:42:30 UTC (rev 248573)
+++ branches/safari-608-branch/Source/WebCore/platform/RuntimeApplicationChecks.h 2019-08-12 23:42:34 UTC (rev 248574)
@@ -43,7 +43,7 @@
bool isInWebProcess();
WEBCORE_EXPORT void setApplicationSDKVersion(uint32_t);
-uint32_t applicationSDKVersion();
+WEBCORE_EXPORT uint32_t applicationSDKVersion();
WEBCORE_EXPORT void setApplicationBundleIdentifier(const String&);
String applicationBundleIdentifier();
@@ -78,6 +78,7 @@
WEBCORE_EXPORT bool isMobileMail();
WEBCORE_EXPORT bool isMobileSafari();
+WEBCORE_EXPORT bool isIMDb();
WEBCORE_EXPORT bool isWebBookmarksD();
WEBCORE_EXPORT bool isDumpRenderTree();
bool isMobileStore();
Modified: branches/safari-608-branch/Source/WebCore/platform/cocoa/RuntimeApplicationChecksCocoa.mm (248573 => 248574)
--- branches/safari-608-branch/Source/WebCore/platform/cocoa/RuntimeApplicationChecksCocoa.mm 2019-08-12 23:42:30 UTC (rev 248573)
+++ branches/safari-608-branch/Source/WebCore/platform/cocoa/RuntimeApplicationChecksCocoa.mm 2019-08-12 23:42:34 UTC (rev 248574)
@@ -208,6 +208,12 @@
return isMobileSafari;
}
+bool IOSApplication::isIMDb()
+{
+ static bool isIMDb = applicationBundleIsEqualTo("com.imdb.imdb"_s);
+ return isIMDb;
+}
+
bool IOSApplication::isWebBookmarksD()
{
static bool isWebBookmarksD = applicationBundleIsEqualTo("com.apple.webbookmarksd"_s);
Modified: branches/safari-608-branch/Source/WebKit/ChangeLog (248573 => 248574)
--- branches/safari-608-branch/Source/WebKit/ChangeLog 2019-08-12 23:42:30 UTC (rev 248573)
+++ branches/safari-608-branch/Source/WebKit/ChangeLog 2019-08-12 23:42:34 UTC (rev 248574)
@@ -1,5 +1,53 @@
2019-08-12 Alan Coon <[email protected]>
+ Cherry-pick r248471. rdar://problem/54130624
+
+ Disable CSSOM View Scrolling API for IMDb iOS app
+ https://bugs.webkit.org/show_bug.cgi?id=200586
+ <rdar://problem/53645833>
+
+ Patch by Alex Christensen <[email protected]> on 2019-08-09
+ Reviewed by Simon Fraser.
+
+ Source/WebCore:
+
+ They are calling scrollHeight on the HTML element and it is running new code introduced in r235806
+ Disable this new feature until they update their app to use the iOS13 SDK.
+
+ * platform/RuntimeApplicationChecks.h:
+ * platform/cocoa/RuntimeApplicationChecksCocoa.mm:
+ (WebCore::IOSApplication::isIMDb):
+
+ Source/WebKit:
+
+ Change the CSSOMViewScrollingAPIEnabled default value to be off for the IMDb app's WKWebViews.
+ I manually verified this is effective in those WKWebViews but no other WKWebViews and that it fixes the radar.
+
+ * Shared/WebPreferences.yaml:
+ * Shared/WebPreferencesDefaultValues.cpp:
+ (WebKit::defaultCSSOMViewScrollingAPIEnabled):
+ * Shared/WebPreferencesDefaultValues.h:
+
+ git-svn-id: https://svn.webkit.org/repository/webkit/trunk@248471 268f45cc-cd09-0410-ab3c-d52691b4dbfc
+
+ 2019-08-09 Alex Christensen <[email protected]>
+
+ Disable CSSOM View Scrolling API for IMDb iOS app
+ https://bugs.webkit.org/show_bug.cgi?id=200586
+ <rdar://problem/53645833>
+
+ Reviewed by Simon Fraser.
+
+ Change the CSSOMViewScrollingAPIEnabled default value to be off for the IMDb app's WKWebViews.
+ I manually verified this is effective in those WKWebViews but no other WKWebViews and that it fixes the radar.
+
+ * Shared/WebPreferences.yaml:
+ * Shared/WebPreferencesDefaultValues.cpp:
+ (WebKit::defaultCSSOMViewScrollingAPIEnabled):
+ * Shared/WebPreferencesDefaultValues.h:
+
+2019-08-12 Alan Coon <[email protected]>
+
Cherry-pick r248469. rdar://problem/54130688
Tapping buttons in Data Detectors lookup previews doesn't work
Modified: branches/safari-608-branch/Source/WebKit/Shared/WebPreferences.yaml (248573 => 248574)
--- branches/safari-608-branch/Source/WebKit/Shared/WebPreferences.yaml 2019-08-12 23:42:30 UTC (rev 248573)
+++ branches/safari-608-branch/Source/WebKit/Shared/WebPreferences.yaml 2019-08-12 23:42:34 UTC (rev 248574)
@@ -1274,7 +1274,7 @@
CSSOMViewScrollingAPIEnabled:
type: bool
- defaultValue: true
+ defaultValue: defaultCSSOMViewScrollingAPIEnabled()
humanReadableName: "CSSOM View Scrolling API"
humanReadableDescription: "Implement standard behavior for scrollLeft, scrollTop, scrollWidth, scrollHeight, scrollTo, scrollBy and scrollingElement."
category: internal
Modified: branches/safari-608-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.cpp (248573 => 248574)
--- branches/safari-608-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.cpp 2019-08-12 23:42:30 UTC (rev 248573)
+++ branches/safari-608-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.cpp 2019-08-12 23:42:34 UTC (rev 248574)
@@ -61,6 +61,15 @@
#endif
}
+bool defaultCSSOMViewScrollingAPIEnabled()
+{
+#if PLATFORM(IOS_FAMILY)
+ if (WebCore::IOSApplication::isIMDb() && applicationSDKVersion() < DYLD_IOS_VERSION_13_0)
+ return false;
+#endif
+ return true;
+}
+
#if ENABLE(TEXT_AUTOSIZING) && !PLATFORM(IOS_FAMILY)
bool defaultTextAutosizingUsesIdempotentMode()
Modified: branches/safari-608-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.h (248573 => 248574)
--- branches/safari-608-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.h 2019-08-12 23:42:30 UTC (rev 248573)
+++ branches/safari-608-branch/Source/WebKit/Shared/WebPreferencesDefaultValues.h 2019-08-12 23:42:34 UTC (rev 248574)
@@ -280,6 +280,7 @@
bool defaultPassiveTouchListenersAsDefaultOnDocument();
bool defaultCustomPasteboardDataEnabled();
+bool defaultCSSOMViewScrollingAPIEnabled();
#if ENABLE(TEXT_AUTOSIZING)
bool defaultTextAutosizingUsesIdempotentMode();
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
