Attachment 371996 Details for Bug 198799 – To land

[patch] To land

bug-198799-20190612160704.patch (text/plain), 2.84 KB, created by Daniel Bates on 2019-06-12 16:07:05 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Daniel Bates

Created: 2019-06-12 16:07:05 PDT

Size: 2.84 KB

patch

obsolete

>Subversion Revision: 246385
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index 2897cbcd42d86e1e5132b3d158d3e3657e880a96..76100eaa4279f0a14da03176dd296e3474a39c77 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,20 @@
>+2019-06-12  Daniel Bates  <dabates@apple.com>
>+
>+        [iOS] Adjust tests platform/ipad/fast/forms/{select-form-run-twice, unfocus-inside-fixed-hittest}.html following r243808
>+        https://bugs.webkit.org/show_bug.cgi?id=198799
>+        <rdar://problem/50181023>
>+
>+        Reviewed by Brent Fulgham.
>+
>+        Following r243808 we no longer defocus a <select> on iPad when its popover is dismissed as a
>+        result of picking a value for it. This is to make the behavior of <select> on iPad like the Mac. 
>+        However the tests platform/ipad/fast/forms/{select-form-run-twice, unfocus-inside-fixed-hittest}.html
>+        depend on the old behavior. So, we need to update them to no longer expect a defocusing of the
>+        <select>/the end of form control interaction.
>+
>+        * platform/ipad/fast/forms/select-form-run-twice.html:
>+        * platform/ipad/fast/forms/unfocus-inside-fixed-hittest.html:
>+
> 2019-06-12  Myles C. Maxfield  <mmaxfield@apple.com>
> 
>         [WHLSL] Educate the property resolver about IndexExpressions
>diff --git a/LayoutTests/platform/ipad/fast/forms/select-form-run-twice.html b/LayoutTests/platform/ipad/fast/forms/select-form-run-twice.html
>index 3d854fd99dfdbabab20cf0196e5bb8f0b6c87300..f330a2c8447bbaf2b56308f271cf4d253cb6ed27 100644
>--- a/LayoutTests/platform/ipad/fast/forms/select-form-run-twice.html
>+++ b/LayoutTests/platform/ipad/fast/forms/select-form-run-twice.html
>@@ -53,9 +53,6 @@
>                 (function() {
>                     uiController.didStartFormControlInteractionCallback = function() {
>                         uiController.selectFormAccessoryPickerRow(${row});
>-                    };
>-
>-                    uiController.didEndFormControlInteractionCallback = function() {
>                         uiController.uiScriptComplete();
>                     };
> 
>diff --git a/LayoutTests/platform/ipad/fast/forms/unfocus-inside-fixed-hittest.html b/LayoutTests/platform/ipad/fast/forms/unfocus-inside-fixed-hittest.html
>index bbc8e33f1ba2e2b3887c04a2103ebe1eb068231f..f9beb24e26d6af076c2b6257b848b0c061c262b3 100644
>--- a/LayoutTests/platform/ipad/fast/forms/unfocus-inside-fixed-hittest.html
>+++ b/LayoutTests/platform/ipad/fast/forms/unfocus-inside-fixed-hittest.html
>@@ -53,9 +53,6 @@
>                 (function() {
>                     uiController.didStartFormControlInteractionCallback = function() {
>                         uiController.selectFormAccessoryPickerRow(5);
>-                    };
>-
>-                    uiController.didEndFormControlInteractionCallback = function() {
>                         uiController.uiScriptComplete();
>                     };
>

Actions: View | Formatted Diff | Diff

Attachments on bug 198799: 371982 | 371996