Attachment 373370 Details for Bug 199257 – Patch

[patch] Patch

bug-199257-20190702163617.patch (text/plain), 5.04 KB, created by Daniel Bates on 2019-07-02 16:36:18 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Daniel Bates

Created: 2019-07-02 16:36:18 PDT

Size: 5.04 KB

patch

obsolete

>Subversion Revision: 246884
>diff --git a/Source/WebKit/ChangeLog b/Source/WebKit/ChangeLog
>index 574dfd24241c644b30e54205c8982286f0c0d213..06bf8f249ecb6ef7baa570c91ca10e87648021b6 100644
>--- a/Source/WebKit/ChangeLog
>+++ b/Source/WebKit/ChangeLog
>@@ -1,3 +1,18 @@
>+2019-07-02  Daniel Bates  <dabates@apple.com>
>+
>+        [iOS] Support select all in non-editable element
>+        https://bugs.webkit.org/show_bug.cgi?id=199257
>+        <rdar://problem/52553667>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Unless we are populating the callout bar always advertise that WebKit can perform Select All.
>+        This way you can press Command + A to select all text even in a non-editable element just like
>+        you can on Mac.
>+
>+        * UIProcess/ios/WKContentViewInteraction.mm:
>+        (-[WKContentView canPerformActionForWebView:withSender:]):
>+
> 2019-07-01  Daniel Bates  <dabates@apple.com>
> 
>         [iOS] Cannot tab cycle through credit card fields on antonsvpatisserie.com checkout page
>diff --git a/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm b/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm
>index 5d0d263aecc00a3505b058b9d82bb010d60ce8b6..fda9c25fc6204708a08e3e3d52fd2a8b51f273e5 100644
>--- a/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm
>+++ b/Source/WebKit/UIProcess/ios/WKContentViewInteraction.mm
>@@ -3053,7 +3053,7 @@ - (BOOL)canPerformActionForWebView:(SEL)action withSender:(id)sender
>         // By platform convention we don't show Select All in the callout menu for a range selection.
>         if ([sender isKindOfClass:UIMenuController.class])
>             return !editorState.selectionIsNone && !editorState.selectionIsRange;
>-        return !editorState.selectionIsNone && self.hasContent;
>+        return YES;
>     }
> 
>     if (action == @selector(replace:))
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index d01261fa55995c3298c0e976d02c7247a913d434..bcc18cffa7b1f84430c16b67eac199ab486303ae 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,16 @@
>+2019-07-02  Daniel Bates  <dabates@apple.com>
>+
>+        [iOS] Support select all in non-editable element
>+        https://bugs.webkit.org/show_bug.cgi?id=199257
>+        <rdar://problem/52553667>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Add a test to ensure that we can perform Select All even when a non-editable element is focused.
>+
>+        * editing/selection/ios/select-all-non-editable-text-using-keyboard-expected.txt: Added.
>+        * editing/selection/ios/select-all-non-editable-text-using-keyboard.html: Added.
>+
> 2019-07-02  Daniel Bates  <dabates@apple.com>
> 
>         Left and right option key has Unidentified key identifier
>diff --git a/LayoutTests/editing/selection/ios/select-all-non-editable-text-using-keyboard-expected.txt b/LayoutTests/editing/selection/ios/select-all-non-editable-text-using-keyboard-expected.txt
>new file mode 100644
>index 0000000000000000000000000000000000000000..30ff7f428f120e7ac1291affb7619cc47ab9b2e9
>--- /dev/null
>+++ b/LayoutTests/editing/selection/ios/select-all-non-editable-text-using-keyboard-expected.txt
>@@ -0,0 +1,10 @@
>+Tests pressing Command + a to select all non-editable text.
>+
>+On success, you will see a series of "PASS" messages, followed by "TEST COMPLETE".
>+
>+
>+PASS window.getSelection().toString() is "Here's to the crazy ones.\n\nThe misfits.\n\nThe rebels."
>+PASS successfullyParsed is true
>+
>+TEST COMPLETE
>+
>diff --git a/LayoutTests/editing/selection/ios/select-all-non-editable-text-using-keyboard.html b/LayoutTests/editing/selection/ios/select-all-non-editable-text-using-keyboard.html
>new file mode 100644
>index 0000000000000000000000000000000000000000..56fb549c78d38083cba13bb5bc4cf4bc1fac9940
>--- /dev/null
>+++ b/LayoutTests/editing/selection/ios/select-all-non-editable-text-using-keyboard.html
>@@ -0,0 +1,46 @@
>+<!DOCTYPE html>
>+<html>
>+<head>
>+<script src="../../../resources/js-test.js"></script>
>+<script src="../../../resources/ui-helper.js"></script>
>+<style>
>+body.hide-everything-except-test-container > :not(#test-container) {
>+    display: none;
>+}
>+</style>
>+</head>
>+<body>
>+<div id="test-container">
>+    <p>Here's to the crazy ones.</p>
>+    <p>The misfits.</p>
>+    <p>The rebels.</p>
>+</div>
>+<script>
>+let testContainer = document.getElementById("test-container");
>+let paragraphs = testContainer.children;
>+let selection = window.getSelection();
>+
>+function toggleOnlyShowTestContainer()
>+{
>+    document.body.classList.toggle("hide-everything-except-test-container");
>+}
>+
>+async function runTest()
>+{
>+    toggleOnlyShowTestContainer();
>+
>+    await UIHelper.callFunctionAndWaitForEvent(() => window.testRunner && UIHelper.keyDown("a", ["metaKey"]) , document, "selectionchange");
>+    shouldBeEqualToString("window.getSelection().toString()", "Here's to the crazy ones.\n\nThe misfits.\n\nThe rebels.");
>+
>+    toggleOnlyShowTestContainer();
>+
>+    document.body.removeChild(document.getElementById("test-container"));
>+    finishJSTest();
>+}
>+
>+window.jsTestIsAsync = true;
>+description("Tests pressing Command + a to select all non-editable text.");
>+runTest();
>+</script>
>+</body>
>+</html>

Actions: View | Formatted Diff | Diff

Attachments on bug 199257: 373368 | 373370 | 373380 | 373428 | 373429