Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1791)

Unified Diff: third_party/WebKit/Source/web/tests/VirtualTimeTest.cpp

Issue 2384033002: reflow comments in web/tests (Closed)
Patch Set: comments (heh!) Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/web/tests/VirtualTimeTest.cpp
diff --git a/third_party/WebKit/Source/web/tests/VirtualTimeTest.cpp b/third_party/WebKit/Source/web/tests/VirtualTimeTest.cpp
index 90a2892d0153ad286f042f7c3db146bef050fd68..06c2754d9773111448a7156ad82f0af191856214 100644
--- a/third_party/WebKit/Source/web/tests/VirtualTimeTest.cpp
+++ b/third_party/WebKit/Source/web/tests/VirtualTimeTest.cpp
@@ -44,9 +44,9 @@ class VirtualTimeTest : public SimTest {
}
void TearDown() override {
- // The SimTest destructor calls runPendingTasks. This is a problem because if there are
- // any repeating tasks, advancing virtual time will cause the runloop to busy loop. Pausing
- // virtual time here fixes that.
+ // The SimTest destructor calls runPendingTasks. This is a problem because
+ // if there are any repeating tasks, advancing virtual time will cause the
+ // runloop to busy loop. Pausing virtual time here fixes that.
webView().scheduler()->setVirtualTimePolicy(
WebViewScheduler::VirtualTimePolicy::PAUSE);
}
@@ -57,7 +57,8 @@ void quitRunLoop() {
base::MessageLoop::current()->QuitNow();
}
-// Some task queues may have repeating v8 tasks that run forever so we impose a hard time limit.
+// Some task queues may have repeating v8 tasks that run forever so we impose a
+// hard time limit.
void runTasksForPeriod(double delayMs) {
Platform::current()->currentThread()->getWebTaskRunner()->postDelayedTask(
BLINK_FROM_HERE, WTF::bind(&quitRunLoop), delayMs);
@@ -162,7 +163,8 @@ TEST_F(VirtualTimeTest,
webView().scheduler()->setVirtualTimePolicy(
WebViewScheduler::VirtualTimePolicy::DETERMINISTIC_LOADING);
- // To ensure determinism virtual time is not allowed to advance until we have seen at least one load.
+ // 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");
« no previous file with comments | « third_party/WebKit/Source/web/tests/TouchActionTest.cpp ('k') | third_party/WebKit/Source/web/tests/VisualViewportTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698