Title: [233533] trunk/LayoutTests
Revision
233533
Author
[email protected]
Date
2018-07-05 12:28:01 -0700 (Thu, 05 Jul 2018)

Log Message

[WK1] editing/spelling/markers.html times out on High Sierra after r233439
https://bugs.webkit.org/show_bug.cgi?id=187351

Reviewed by Tim Horton.

This is currently timing out on High Sierra, because the `verifyUnexpectedMarkers` portion of the
test fails to actually make use of the override spell checking results. Fix this by passing the
results when calling the function.

* editing/spelling/markers.html:

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (233532 => 233533)


--- trunk/LayoutTests/ChangeLog	2018-07-05 19:11:39 UTC (rev 233532)
+++ trunk/LayoutTests/ChangeLog	2018-07-05 19:28:01 UTC (rev 233533)
@@ -1,3 +1,16 @@
+2018-07-05  Wenson Hsieh  <[email protected]>
+
+        [WK1] editing/spelling/markers.html times out on High Sierra after r233439
+        https://bugs.webkit.org/show_bug.cgi?id=187351
+
+        Reviewed by Tim Horton.
+
+        This is currently timing out on High Sierra, because the `verifyUnexpectedMarkers` portion of the
+        test fails to actually make use of the override spell checking results. Fix this by passing the
+        results when calling the function.
+
+        * editing/spelling/markers.html:
+
 2018-07-05  Nan Wang  <[email protected]>
 
         AX: setValue on text controls should send out key events

Modified: trunk/LayoutTests/editing/spelling/markers.html (233532 => 233533)


--- trunk/LayoutTests/editing/spelling/markers.html	2018-07-05 19:11:39 UTC (rev 233532)
+++ trunk/LayoutTests/editing/spelling/markers.html	2018-07-05 19:28:01 UTC (rev 233533)
@@ -81,8 +81,8 @@
     function() { verifyDesiredMarkers(elementWithGrammarAndSpellingIssue, misspellings.slice(4, 6)) },
 
     // Those expect to have only one kind of markers either spelling or grammar.
-    function() { verifyUnexpectedMarkers(elementWithGrammarIssue, misspellings.slice(0, 1)), results.slice(0, 1) },
-    function() { verifyUnexpectedMarkers(elementWithGrammarIssue, misspellings.slice(1, 2)), results.slice(1, 2) },
+    function() { verifyUnexpectedMarkers(elementWithGrammarIssue, misspellings.slice(0, 1), results.slice(0, 1)) },
+    function() { verifyUnexpectedMarkers(elementWithGrammarIssue, misspellings.slice(1, 2), results.slice(1, 2)) },
     function() { verifyUnexpectedMarkers(elementWithSpellingIssue, misspellings.slice(3, 4)) },
 ];
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to