Attachment 346727 Details for Bug 188389 – Patch

[patch] Patch

bug-188389-20180807140117.patch (text/plain), 1.67 KB, created by Sihui Liu on 2018-08-07 14:01:18 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Sihui Liu

Created: 2018-08-07 14:01:18 PDT

Size: 1.67 KB

patch

obsolete

>Subversion Revision: 234635
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 2e6ffe1072a052e26eb56a508ecf212e5b14c8ba..16179af963404f0df8e309854cde18a4a398a2ec 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,15 @@
>+2018-08-07  Sihui Liu  <sihui_liu@apple.com>
>+
>+        Avoid timeout resulted from calling waitUntilDone when test is not running
>+        https://bugs.webkit.org/show_bug.cgi?id=188389
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        If test is not running, we should not set the waitUntilDone flag, or it could make subsequent tests timeout.
>+
>+        * WebKitTestRunner/InjectedBundle/TestRunner.cpp:
>+        (WTR::TestRunner::waitUntilDone):
>+
> 2018-08-06  David Quesada  <david_quesada@apple.com>
> 
>         webkitdirs.pm should default to iPhone SE for 64-bit testing
>diff --git a/Tools/WebKitTestRunner/InjectedBundle/TestRunner.cpp b/Tools/WebKitTestRunner/InjectedBundle/TestRunner.cpp
>index 27f0dd285b540ecff45825c63ee9dfd5972ce95a..43de052600d82fe53e14c353791034dac06669b3 100644
>--- a/Tools/WebKitTestRunner/InjectedBundle/TestRunner.cpp
>+++ b/Tools/WebKitTestRunner/InjectedBundle/TestRunner.cpp
>@@ -185,9 +185,13 @@ void TestRunner::waitUntilDownloadFinished()
> 
> void TestRunner::waitUntilDone()
> {
>+    auto& injectedBundle = InjectedBundle::singleton();
>+    if (!injectedBundle.isTestRunning())
>+        return;
>+
>     setWaitUntilDone(true);
>     // FIXME: Watchdog timer should be moved to UI process in order to take the elapsed time in anotehr process into account.
>-    if (InjectedBundle::singleton().useWaitToDumpWatchdogTimer())
>+    if (injectedBundle.useWaitToDumpWatchdogTimer())
>         initializeWaitToDumpWatchdogTimerIfNeeded();
> }
>

Actions: View | Formatted Diff | Diff

Attachments on bug 188389: 346727 | 346750 | 346850 | 346854 | 346856 | 347039