[iOS] Normalize test result of fast/events/ios/keyboard-event-key-attribute.html...
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Jun 2019 23:01:57 +0000 (23:01 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 13 Jun 2019 23:01:57 +0000 (23:01 +0000)
https://bugs.webkit.org/show_bug.cgi?id=198835
<rdar://problem/46082708>

Reviewed by Darin Adler.

WebKit now knows how to handle flag change events on iOS and dispatch DOM events just like Mac.
Until USE(UIKIT_KEYBOARD_ADDITIONS) is enabled by default WebKit does not expect to receive
modifier flag changes. But once this compile time flag is enabled it will and the test
fast/events/ios/keyboard-event-key-attribute.html will fail because it types '@' and only
expects to see a single keydown for '@' even though it actually requires two key downs to
generate an '@': Shift, and 2 (on a US keyboard layout). We have a bunch of other tests to
check that modifier key presses generate DOM events. So, let's just normalize the result of
this test to ignore Shift keydowns.

* fast/events/ios/keyboard-event-key-attribute.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@246417 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/fast/events/ios/keyboard-event-key-attribute.html

index 41ca596..f8d2e0a 100644 (file)
@@ -1,3 +1,22 @@
+2019-06-13  Daniel Bates  <dabates@apple.com>
+
+        [iOS] Normalize test result of fast/events/ios/keyboard-event-key-attribute.html to handle possible modifier dispatch
+        https://bugs.webkit.org/show_bug.cgi?id=198835
+        <rdar://problem/46082708>
+
+        Reviewed by Darin Adler.
+
+        WebKit now knows how to handle flag change events on iOS and dispatch DOM events just like Mac.
+        Until USE(UIKIT_KEYBOARD_ADDITIONS) is enabled by default WebKit does not expect to receive
+        modifier flag changes. But once this compile time flag is enabled it will and the test
+        fast/events/ios/keyboard-event-key-attribute.html will fail because it types '@' and only
+        expects to see a single keydown for '@' even though it actually requires two key downs to
+        generate an '@': Shift, and 2 (on a US keyboard layout). We have a bunch of other tests to
+        check that modifier key presses generate DOM events. So, let's just normalize the result of
+        this test to ignore Shift keydowns.
+
+        * fast/events/ios/keyboard-event-key-attribute.html:
+
 2019-06-13  Truitt Savell  <tsavell@apple.com>
 
         Test gardening after r246406
index 7c1a2fc..58750af 100644 (file)
@@ -30,6 +30,9 @@ function getTypingUIScript(x, y)
 var i = 0;
 var testInput = document.getElementById("testInput");
 testInput.onkeydown = function(e) {
+    // Ignore the Shift key so as to normalize the test results across different iOS versions.
+    if (e.key == "Shift")
+        return;
     event = e;
     shouldBeEqualToString("event.key", "" + tests[i][0]);
     shouldBeEqualToString("event.code", "" + tests[i][1]);