Change VirtualTimePolicy::PAUSE_IF_NETWORK_FETCHES_PENDING

third_party/WebKit/Source/web/tests/VirtualTimeTest.cpp

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "platform/testing/UnitTestHelpers.h"
 #include "public/platform/Platform.h"
 #include "public/platform/WebViewScheduler.h"
 #include "public/web/WebLocalFrame.h"
 #include "public/web/WebScriptExecutionCallback.h"
 #include "public/web/WebScriptSource.h"
@@ skipping 20 matching lines @@
 };
 } // namespace
 
 class VirtualTimeTest : public SimTest {
 protected:
     String ExecuteJavaScript(String scriptSource)
     {
         ScriptExecutionCallbackHelper callbackHelper;
         webView().mainFrame()->toWebLocalFrame()->requestExecuteScriptAndReturnValue(
             WebScriptSource(WebString(scriptSource)), false, &callbackHelper);
-        testing::runPendingTasks();
         return callbackHelper.result();
     }
 };
 
+namespace {
+    void quitRunLoop()
+    {
+        base::MessageLoop::current()->QuitNow();
+    }
+
+    // Some task queues may have repeating tasks that run forever so we impose a hard time limit.

[Sami, 2016/07/29 14:05:51]

"repeating v8 tasks" to make it clear it's not the test itself generating these
tasks?

[alex clarke (OOO till 29th), 2016/07/29 14:42:16]

Done.

+    void runTasksForPeriod(double delayMs)
+    {
+        Platform::current()->currentThread()->getWebTaskRunner()->postDelayedTask(BLINK_FROM_HERE, WTF::bind(&quitRunLoop), delayMs);
+        testing::enterRunLoop();
+    }
+}
+
 TEST_F(VirtualTimeTest, DOMTimersFireInExpectedOrder)
 {
     webView().scheduler()->enableVirtualTime();
 
     ExecuteJavaScript(
         "var run_order = [];"
         "function timerFn(delay, value) {"
         "  setTimeout(function() { run_order.push(value); }, delay);"
         "};"
         "var one_hour = 60 * 60 * 1000;"
         "timerFn(one_hour * 100, 'a');"
         "timerFn(one_hour * 10, 'b');"
         "timerFn(one_hour, 'c');");
 
     // Normally the JS runs pretty much instantly but the timer callbacks will
     // take 100h to fire, but thanks to timer fast forwarding we can make them
     // fire immediatly.
 
+    testing::runPendingTasks();
     EXPECT_EQ("c, b, a", ExecuteJavaScript("run_order.join(', ')"));
 }
 
 TEST_F(VirtualTimeTest, SetInterval)
 {
     webView().scheduler()->enableVirtualTime();
 
     ExecuteJavaScript(
         "var run_order = [];"
         "var count = 10;"
         "var interval_handle = setInterval(function() {"
         "  if (--window.count == 0) {"
         "     clearInterval(interval_handle);"
         "  }"
         "  run_order.push(count);"
         "}, 1000);"
         "setTimeout(function() { run_order.push('timer'); }, 1500);");
 
-    // If virtual time is not supplied to TimerBase then the setInterval
-    // won't fire 10x.
+    runTasksForPeriod(12000);
+
     EXPECT_EQ("9, timer, 8, 7, 6, 5, 4, 3, 2, 1, 0", ExecuteJavaScript("run_order.join(', ')"));
 }
 
 TEST_F(VirtualTimeTest, AllowVirtualTimeToAdvance)
 {
     webView().scheduler()->enableVirtualTime();
     webView().scheduler()->setVirtualTimePolicy(WebViewScheduler::VirtualTimePolicy::PAUSE);
 
     ExecuteJavaScript(
         "var run_order = [];"
         "timerFn = function(delay, value) {"
         "  setTimeout(function() { run_order.push(value); }, delay);"
         "};"
         "timerFn(100, 'a');"
         "timerFn(10, 'b');"
         "timerFn(1, 'c');");
 
+    testing::runPendingTasks();
     EXPECT_EQ("", ExecuteJavaScript("run_order.join(', ')"));
 
     webView().scheduler()->setVirtualTimePolicy(WebViewScheduler::VirtualTimePolicy::ADVANCE);
-    testing::runPendingTasks();
+    runTasksForPeriod(1000);
 
     EXPECT_EQ("c, b, a", ExecuteJavaScript("run_order.join(', ')"));
 }
 
 TEST_F(VirtualTimeTest, VirtualTimeNotAllowedToAdvanceWhileResourcesLoading)
 {
     webView().scheduler()->enableVirtualTime();
     webView().scheduler()->setVirtualTimePolicy(WebViewScheduler::VirtualTimePolicy::PAUSE_IF_NETWORK_FETCHES_PENDING);
 
-    EXPECT_TRUE(webView().scheduler()->virtualTimeAllowedToAdvance());
+    // To ensure determinism virtual time is not allowed to advance until we have seen at least one load.
+    EXPECT_FALSE(webView().scheduler()->virtualTimeAllowedToAdvance());
 
     SimRequest mainResource("https://example.com/test.html", "text/html");
     SimRequest cssResource("https://example.com/test.css", "text/css");
 
-    // Not loading, virtual time should be able to advance.
-    EXPECT_TRUE(webView().scheduler()->virtualTimeAllowedToAdvance());
-
     // Loading, virtual time should not advance.
     loadURL("https://example.com/test.html");
     EXPECT_FALSE(webView().scheduler()->virtualTimeAllowedToAdvance());
 
     mainResource.start();
 
     // Still Loading, virtual time should not advance.
     mainResource.write("<!DOCTYPE html><link rel=stylesheet href=test.css>");
     EXPECT_FALSE(webView().scheduler()->virtualTimeAllowedToAdvance());
 
     // Still Loading, virtual time should not advance.
     cssResource.start();
     cssResource.write("a { color: red; }");
     EXPECT_FALSE(webView().scheduler()->virtualTimeAllowedToAdvance());
 
     // Still Loading, virtual time should not advance.
     cssResource.finish();
     EXPECT_FALSE(webView().scheduler()->virtualTimeAllowedToAdvance());
 
     // Still Loading, virtual time should not advance.
     mainResource.write("<body>");
     EXPECT_FALSE(webView().scheduler()->virtualTimeAllowedToAdvance());
 
     // Finished loading, virtual time should be able to advance.
     mainResource.finish();
     EXPECT_TRUE(webView().scheduler()->virtualTimeAllowedToAdvance());
 }
 
 } // namespace blink