Attachment 372077 Details for Bug 198842 – Patch

[patch] Patch

bug-198842-20190613134801.patch (text/plain), 11.91 KB, created by Megan Gardner on 2019-06-13 13:48:01 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Megan Gardner

Created: 2019-06-13 13:48:01 PDT

Size: 11.91 KB

patch

obsolete

>Subversion Revision: 246375
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 0a678a3271016c385ef98bc5dd738cac32ac2679..c8e29096f4d15ba05a838b9341d413cee0e4ac33 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,21 @@
>+2019-06-13  Megan Gardner  <megan_gardner@apple.com>
>+
>+        Move Synthetic Editing Commands to behind an experimental feature flag
>+        https://bugs.webkit.org/show_bug.cgi?id=198842
>+        <rdar://problem/50594700>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Moving from a quirk to a feature flag.
>+
>+        * page/Quirks.cpp:
>+        (WebCore::shouldEmulateEditingButtonsAndGesturesInHiddenEditableAreasForHost): Deleted.
>+        (WebCore::Quirks::shouldEmulateEditingButtonsAndGesturesInHiddenEditableAreas const): Deleted.
>+        * page/Quirks.h:
>+        * page/RuntimeEnabledFeatures.h:
>+        (WebCore::RuntimeEnabledFeatures::setSyntheticEditingCommandsEnabled):
>+        (WebCore::RuntimeEnabledFeatures::syntheticEditingCommandsEnabled const):
>+
> 2019-06-12  Sam Weinig  <weinig@apple.com>
> 
>         Remove dead code in user agent construction
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index baf2ae92aec8ddcb6db2e4315058df8d6104cf50..875a2b08ba9b740cb87a4862b1bd1ab03da9a4cc 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-06-13  Megan Gardner  <megan_gardner@apple.com>
>+
>+        Move Synthetic Editing Commands to behind an experimental feature flag
>+        https://bugs.webkit.org/show_bug.cgi?id=198842
>+        <rdar://problem/50594700>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add a feature flag to gate synthetic editing commands.
>+
>+        * Shared/WebPreferences.yaml:
>+        * WebProcess/WebPage/ios/WebPageIOS.mm:
>+        (WebKit::WebPage::getFocusedElementInformation):
>+
> 2019-06-12  Ludovico de Nittis  <ludovico.denittis@collabora.com>
> 
>         [GTK] GTK_STOCK_* types have been deprecated since GTK 3.10
>diff --git a/Source/WebKitLegacy/mac/ChangeLog b/Source/WebKitLegacy/mac/ChangeLog
>index b95d7bb1be74a51602e9409f8c804a64fe29276b..8cc6943e2797b7503ca30bf90db578bddcdee394 100644
>--- a/Source/WebKitLegacy/mac/ChangeLog
>+++ b/Source/WebKitLegacy/mac/ChangeLog
>@@ -1,3 +1,22 @@
>+2019-06-13  Megan Gardner  <megan_gardner@apple.com>
>+
>+        Move Synthetic Editing Commands to behind an experimental feature flag
>+        https://bugs.webkit.org/show_bug.cgi?id=198842
>+        <rdar://problem/50594700>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add plumbing for synthetic editing command feature flag.
>+
>+        * WebView/WebPreferenceKeysPrivate.h:
>+        * WebView/WebPreferences.mm:
>+        (+[WebPreferences initialize]):
>+        (-[WebPreferences syntheticEditingCommandsEnabled]):
>+        (-[WebPreferences setSyntheticEditingCommandsEnabled:]):
>+        * WebView/WebPreferencesPrivate.h:
>+        * WebView/WebView.mm:
>+        (-[WebView _preferencesChanged:]):
>+
> 2019-06-10  Sam Weinig  <weinig@apple.com>
> 
>         Remove Dashboard support
>diff --git a/Source/WebCore/page/Quirks.cpp b/Source/WebCore/page/Quirks.cpp
>index 2e99641d84b4da8e62521b8ec9ee5b5eeffdaafd..6543984a3afe6c5d3f5eb9ec6b2c17048cd0fe01 100644
>--- a/Source/WebCore/page/Quirks.cpp
>+++ b/Source/WebCore/page/Quirks.cpp
>@@ -242,16 +242,6 @@ static bool shouldSuppressAutocorrectionAndAutocaptializationInHiddenEditableAre
> #endif
> }
> 
>-static bool shouldEmulateEditingButtonsAndGesturesInHiddenEditableAreasForHost(const StringView& host)
>-{
>-#if PLATFORM(IOS_FAMILY)
>-    return equalLettersIgnoringASCIICase(host, "docs.google.com");
>-#else
>-    UNUSED_PARAM(host);
>-    return false;
>-#endif
>-}
>-
> bool Quirks::shouldDispatchSyntheticMouseEventsWhenModifyingSelection() const
> {
>     if (m_document->settings().shouldDispatchSyntheticMouseEventsWhenModifyingSelection())
>@@ -270,14 +260,6 @@ bool Quirks::shouldDispatchSyntheticMouseEventsWhenModifyingSelection() const
>     return false;
> }
> 
>-bool Quirks::shouldEmulateEditingButtonsAndGesturesInHiddenEditableAreas() const
>-{
>-    if (!needsQuirks())
>-        return false;
>-
>-    return shouldEmulateEditingButtonsAndGesturesInHiddenEditableAreasForHost(m_document->topDocument().url().host());
>-}
>-
> bool Quirks::shouldSuppressAutocorrectionAndAutocaptializationInHiddenEditableAreas() const
> {
>     if (!needsQuirks())
>diff --git a/Source/WebCore/page/Quirks.h b/Source/WebCore/page/Quirks.h
>index 0235733ab11e4cb3761404b25c9b3b1c1bca8004..cbd2fd23e6878a1139bb65199f00c3362e3735b2 100644
>--- a/Source/WebCore/page/Quirks.h
>+++ b/Source/WebCore/page/Quirks.h
>@@ -58,7 +58,6 @@ public:
> 
>     WEBCORE_EXPORT bool shouldDispatchSyntheticMouseEventsWhenModifyingSelection() const;
>     WEBCORE_EXPORT bool shouldSuppressAutocorrectionAndAutocaptializationInHiddenEditableAreas() const;
>-    WEBCORE_EXPORT bool shouldEmulateEditingButtonsAndGesturesInHiddenEditableAreas() const;
>     WEBCORE_EXPORT bool isTouchBarUpdateSupressedForHiddenContentEditable() const;
>     WEBCORE_EXPORT bool isNeverRichlyEditableForTouchBar() const;
> 
>diff --git a/Source/WebCore/page/RuntimeEnabledFeatures.h b/Source/WebCore/page/RuntimeEnabledFeatures.h
>index 1b9ddaccb8cf3bcdafd70904688cfee466944599..cb200e513d3b9d4e3adb572d04818f18d45b4000 100644
>--- a/Source/WebCore/page/RuntimeEnabledFeatures.h
>+++ b/Source/WebCore/page/RuntimeEnabledFeatures.h
>@@ -183,6 +183,9 @@ public:
> 
>     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; }
> 
> #if ENABLE(LAYOUT_FORMATTING_CONTEXT)
>     void setLayoutFormattingContextEnabled(bool isEnabled) { m_layoutFormattingContextEnabled = isEnabled; }
>@@ -411,6 +414,7 @@ private:
>     bool m_webAPIStatisticsEnabled { false };
>     bool m_CSSCustomPropertiesAndValuesEnabled { false };
>     bool m_pointerEventsEnabled { true };
>+    bool m_syntheticEditingCommandsEnabled { true };
>     bool m_webSQLEnabled { true };
>     bool m_pageAtRuleSupportEnabled { false };
> 
>diff --git a/Source/WebKit/Shared/WebPreferences.yaml b/Source/WebKit/Shared/WebPreferences.yaml
>index 5ec408bf75c7ad61cfdd98c99ebfe196d7f64951..3009c14098fec5cd6f316e8ccc5456bdb51803d4 100644
>--- a/Source/WebKit/Shared/WebPreferences.yaml
>+++ b/Source/WebKit/Shared/WebPreferences.yaml
>@@ -1254,6 +1254,14 @@ PointerEventsEnabled:
>   webcoreBinding: RuntimeEnabledFeatures
>   category: experimental
> 
>+SyntheticEditingCommandsEnabled:
>+  type: bool
>+  defaultValue: true
>+  humanReadableName: "Synthetic Editing Commands"
>+  humanReadableDescription: "Enable Synthetic Editing Commands"
>+  webcoreBinding: RuntimeEnabledFeatures
>+  category: experimental
>+
> CSSOMViewScrollingAPIEnabled:
>   type: bool
>   defaultValue: true
>diff --git a/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm b/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm
>index 0887027410c6ffd4534fbbda7ddc96315150ecd7..356175bca764b3ed97d009b1146aff88197fb464 100644
>--- a/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm
>+++ b/Source/WebKit/WebProcess/WebPage/ios/WebPageIOS.mm
>@@ -116,6 +116,7 @@
> #import <WebCore/RenderThemeIOS.h>
> #import <WebCore/RenderView.h>
> #import <WebCore/RuntimeApplicationChecks.h>
>+#import <WebCore/RuntimeEnabledFeatures.h>
> #import <WebCore/Settings.h>
> #import <WebCore/SharedBuffer.h>
> #import <WebCore/StyleProperties.h>
>@@ -2937,7 +2938,7 @@ void WebPage::getFocusedElementInformation(FocusedElementInformation& informatio
>             information.isAutocorrect = focusedElement.shouldAutocorrect();
>             information.autocapitalizeType = focusedElement.autocapitalizeType();
>             information.inputMode = focusedElement.canonicalInputMode();
>-            information.shouldSynthesizeKeyEventsForEditing = focusedElement.document().quirks().shouldEmulateEditingButtonsAndGesturesInHiddenEditableAreas();
>+            information.shouldSynthesizeKeyEventsForEditing = RuntimeEnabledFeatures::sharedFeatures().syntheticEditingCommandsEnabled();
>         } else {
>             information.isAutocorrect = true;
>             information.autocapitalizeType = AutocapitalizeTypeDefault;
>diff --git a/Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h b/Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h
>index 3b21e6405e224d71a0d43ad4615da91d66fcf295..86addf6a11b5efa3e73a0f5c20a368a54acceeaf 100644
>--- a/Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h
>+++ b/Source/WebKitLegacy/mac/WebView/WebPreferenceKeysPrivate.h
>@@ -54,6 +54,7 @@
> #define WebKitJavaScriptMarkupEnabledPreferenceKey @"WebKitJavaScriptMarkupEnabled"
> #define WebKitWebAnimationsEnabledPreferenceKey @"WebKitWebAnimationsEnabled"
> #define WebKitPointerEventsEnabledPreferenceKey @"WebKitPointerEventsEnabled"
>+#define WebKitSyntheticEditingCommandsEnabledPreferenceKey @"WebKitSyntheticEditingCommandsEnabled"
> #define WebKitWebSecurityEnabledPreferenceKey @"WebKitWebSecurityEnabled"
> #define WebKitAllowUniversalAccessFromFileURLsPreferenceKey @"WebKitAllowUniversalAccessFromFileURLs"
> #define WebKitAllowFileAccessFromFileURLsPreferenceKey @"WebKitAllowFileAccessFromFileURLs"
>diff --git a/Source/WebKitLegacy/mac/WebView/WebPreferences.mm b/Source/WebKitLegacy/mac/WebView/WebPreferences.mm
>index 81877304a01d9a9f6823c871c30341e2d9397647..ec9132769148fa7008b1c49b2718ccd26244b08d 100644
>--- a/Source/WebKitLegacy/mac/WebView/WebPreferences.mm
>+++ b/Source/WebKitLegacy/mac/WebView/WebPreferences.mm
>@@ -643,6 +643,7 @@ + (void)initialize
>         [NSNumber numberWithBool:NO], WebKitDirectoryUploadEnabledPreferenceKey,
>         [NSNumber numberWithBool:YES], WebKitWebAnimationsEnabledPreferenceKey,
>         [NSNumber numberWithBool:YES], WebKitPointerEventsEnabledPreferenceKey,
>+        [NSNumber numberWithBool:YES], WebKitSyntheticEditingCommandsEnabledPreferenceKey,
> 
> #if PLATFORM(IOS_FAMILY)
>         @NO, WebKitVisualViewportAPIEnabledPreferenceKey,
>@@ -3156,6 +3157,16 @@ - (void)setPointerEventsEnabled:(BOOL)flag
>     [self _setBoolValue:flag forKey:WebKitPointerEventsEnabledPreferenceKey];
> }
> 
>+- (BOOL)syntheticEditingCommandsEnabled
>+{
>+    return [self _boolValueForKey:WebKitPointerEventsEnabledPreferenceKey];
>+}
>+
>+- (void)setSyntheticEditingCommandsEnabled:(BOOL)flag
>+{
>+    [self _setBoolValue:flag forKey:WebKitPointerEventsEnabledPreferenceKey];
>+}
>+
> - (BOOL)fetchAPIKeepAliveEnabled
> {
>     return [self _boolValueForKey:WebKitFetchAPIEnabledPreferenceKey];
>diff --git a/Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h b/Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h
>index 1229edf9e6d6b2d4d1cb731b14168f58f0f17175..a8b9ac7206268f4260acb996c9d6d7eab093bd99 100644
>--- a/Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h
>+++ b/Source/WebKitLegacy/mac/WebView/WebPreferencesPrivate.h
>@@ -573,6 +573,9 @@ extern NSString *WebPreferencesCacheModelChangedInternalNotification WEBKIT_DEPR
> - (void)setPointerEventsEnabled:(BOOL)flag;
> - (BOOL)pointerEventsEnabled;
> 
>+- (void)setSyntheticEditingCommandsEnabled:(BOOL)flag;
>+- (BOOL)syntheticEditingCommandsEnabled;
>+
> - (void)setFetchAPIKeepAliveEnabled:(BOOL)flag;
> - (BOOL)fetchAPIKeepAliveEnabled;
> 
>diff --git a/Source/WebKitLegacy/mac/WebView/WebView.mm b/Source/WebKitLegacy/mac/WebView/WebView.mm
>index 38de68b3e0e9dda18edccbdd456117211a9f4602..5926cbd2608753110fa834d486f7043855c3acfc 100644
>--- a/Source/WebKitLegacy/mac/WebView/WebView.mm
>+++ b/Source/WebKitLegacy/mac/WebView/WebView.mm
>@@ -3160,6 +3160,8 @@ - (void)_preferencesChanged:(WebPreferences *)preferences
> #if ENABLE(POINTER_EVENTS)
>     RuntimeEnabledFeatures::sharedFeatures().setPointerEventsEnabled([preferences pointerEventsEnabled]);
> #endif
>+    
>+    RuntimeEnabledFeatures::sharedFeatures().setSyntheticEditingCommandsEnabled([preferences syntheticEditingCommandsEnabled]);
> 
> #if ENABLE(INTERSECTION_OBSERVER)
>     RuntimeEnabledFeatures::sharedFeatures().setIntersectionObserverEnabled(preferences.intersectionObserverEnabled);

Actions: View | Formatted Diff | Diff

Attachments on bug 198842: 372077 | 372081 | 372087 | 372137