REGRESSION (r201263): Some tests have become flaky timeouts.
authorap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 4 Jun 2016 19:11:17 +0000 (19:11 +0000)
committerap@apple.com <ap@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 4 Jun 2016 19:11:17 +0000 (19:11 +0000)
https://bugs.webkit.org/show_bug.cgi?id=157990

Reviewed by Darin Adler.

Tools:

* WebKitTestRunner/TestInvocation.cpp: (WTR::TestInvocation::invoke):
It is somewhat legitimate for WebContent to not respond to messages after the test
is done; it can be e.g. re-doing a costly repaint. So, we should restart is silently,
without making the current test fail.

LayoutTests:

* platform/mac-wk2/TestExpectations:
* platform/mac/TestExpectations:

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

LayoutTests/ChangeLog
LayoutTests/platform/mac-wk2/TestExpectations
LayoutTests/platform/mac/TestExpectations
Tools/ChangeLog
Tools/WebKitTestRunner/TestInvocation.cpp

index 1c2555b..26d8a1e 100644 (file)
@@ -1,3 +1,13 @@
+2016-06-04  Alexey Proskuryakov  <ap@apple.com>
+
+        REGRESSION (r201263): Some tests have become flaky timeouts.
+        https://bugs.webkit.org/show_bug.cgi?id=157990
+
+        Reviewed by Darin Adler.
+
+        * platform/mac-wk2/TestExpectations:
+        * platform/mac/TestExpectations:
+
 2016-06-04  Joseph Pecoraro  <pecoraro@apple.com>
 
         Add better basic isNaN test coverage
index 7043e99..5f64c05 100644 (file)
@@ -462,6 +462,4 @@ webkit.org/b/155757 [ Debug ElCapitan ] media/media-document-audio-repaint.html
 
 webkit.org/b/157589 fast/text-autosizing/ios/text-autosizing-after-back.html [ Pass Timeout ]
 
-webkit.org/b/158133 [ Debug ] js/function-apply.html [ Pass Timeout ]
-
 webkit.org/b/158237 fast/scrolling/scroll-container-horizontally.html [ Pass Timeout ]
index 9e79cb0..1d01396 100644 (file)
@@ -1344,8 +1344,6 @@ webkit.org/b/156320 fast/regions/inline-block-inside-anonymous-overflow.html [ I
 
 webkit.org/b/155196 media/video-with-file-url-allowed-by-csp-media-src-star-with-AllowContentSecurityPolicySourceStarToMatchAnyProtocol-enabled.html [ Skip ]
 
-webkit.org/b/157990 [ Release ] mathml/wbr-in-mroot-crash.html [ Timeout ]
-
 webkit.org/b/158101 imported/blink/http/tests/plugins/get-url-notify-on-removal.html [ Pass Timeout ]
 
 webkit.org/b/143653 [ Yosemite ] http/tests/websocket/tests/hybi/upgrade-simple-ws.html [ Skip ] # Timeout
index 0649f22..52f05a6 100644 (file)
@@ -1,3 +1,15 @@
+2016-06-04  Alexey Proskuryakov  <ap@apple.com>
+
+        REGRESSION (r201263): Some tests have become flaky timeouts.
+        https://bugs.webkit.org/show_bug.cgi?id=157990
+
+        Reviewed by Darin Adler.
+
+        * WebKitTestRunner/TestInvocation.cpp: (WTR::TestInvocation::invoke):
+        It is somewhat legitimate for WebContent to not respond to messages after the test
+        is done; it can be e.g. re-doing a costly repaint. So, we should restart is silently,
+        without making the current test fail.
+
 2016-06-03  Youenn Fablet  <youenn.fablet@crf.canon.fr>
 
         [GTK] Update YouCompleteMe script to make all include paths absolute
index d86e8e1..e71bf08 100644 (file)
@@ -174,8 +174,6 @@ end:
         dumpWebProcessUnresponsiveness();
     else if (TestController::singleton().resetStateToConsistentValues(m_options))
         return;
-    else
-        dumpWebProcessUnresponsiveness("TestController::resetStateToConsistentValues failed\n");
 
     // The process is unresponsive, so let's start a new one.
     TestController::singleton().terminateWebContentProcess();