Attachment 371885 Details for Bug 198768 – Patch

[patch] Patch

bug-198768-20190611151347.patch (text/plain), 2.02 KB, created by Daniel Bates on 2019-06-11 15:13:47 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Daniel Bates

Created: 2019-06-11 15:13:47 PDT

Size: 2.02 KB

patch

obsolete

>Subversion Revision: 246325
>diff --git a/LayoutTests/ChangeLog b/LayoutTests/ChangeLog
>index 45ab3d24e4088da28a5fe142f2dc2523541e8ac3..ad9bc6f96b8fe281eeda1a3c238bed170d713dd8 100644
>--- a/LayoutTests/ChangeLog
>+++ b/LayoutTests/ChangeLog
>@@ -1,3 +1,16 @@
>+2019-06-11  Daniel Bates  <dabates@apple.com>
>+
>+        [iOS] Adjust test fast/events/ios/keyup.html to ignore Shift keyups
>+        https://bugs.webkit.org/show_bug.cgi?id=198768
>+        <rdar://problem/46082743>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        Ignore Shift keyups as in the future they will be emitted. We have existing test coverage for them
>+        currently skipped in OpenSource that will be unskipped in the future once UIKit support is available. 
>+
>+        * fast/events/ios/keyup.html:
>+
> 2019-06-11  Daniel Bates  <dabates@apple.com>
> 
>         Import Content Security Policy Web Platform Tests
>diff --git a/LayoutTests/fast/events/ios/keyup.html b/LayoutTests/fast/events/ios/keyup.html
>index 5f0a9b9a82fd562762fbb7818e3d0a48df373b86..3ae87158cd0bf59963f7ee9404d67c39e7aac84e 100644
>--- a/LayoutTests/fast/events/ios/keyup.html
>+++ b/LayoutTests/fast/events/ios/keyup.html
>@@ -26,10 +26,15 @@ function log(message)
> 
> function logKeyEvent(event)
> {
>-    let pieces = [];
>-    for (let propertyName of ["type", "key", "code", "keyIdentifier", "keyCode", "charCode", "keyCode", "which"])
>-        pieces.push(`${propertyName}: ${event[propertyName]}`);
>-    log(pieces.join(", "));
>+    // Ignore Shift keyups, if they are dispatched, as not all versions of iOS dispatch them.
>+    if (event.key == "Shift")
>+        ++numberOfFiredKeyEventsForTestComplete;
>+    if (event.key !== "Shift") {
>+        let pieces = [];
>+        for (let propertyName of ["type", "key", "code", "keyIdentifier", "keyCode", "charCode", "keyCode", "which"])
>+            pieces.push(`${propertyName}: ${event[propertyName]}`);
>+        log(pieces.join(", "));
>+    }
>     if (++numberOfFiredKeyEvents >= numberOfFiredKeyEventsForTestComplete && window.testRunner)
>         testRunner.notifyDone();
> }

Actions: View | Formatted Diff | Diff

Attachments on bug 198768: 371885 | 371888