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

Unified Diff: third_party/WebKit/Source/web/tests/PrerenderingTest.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/PrerenderingTest.cpp
diff --git a/third_party/WebKit/Source/web/tests/PrerenderingTest.cpp b/third_party/WebKit/Source/web/tests/PrerenderingTest.cpp
index 150ff56f4cd340bff0e7366a3dc9fbe03a534937..4ef87aa50ba662560c3182effa8a4196450ca903 100644
--- a/third_party/WebKit/Source/web/tests/PrerenderingTest.cpp
+++ b/third_party/WebKit/Source/web/tests/PrerenderingTest.cpp
@@ -281,7 +281,8 @@ TEST_F(PrerenderingTest, AbandonPrerender) {
EXPECT_EQ(1u, prerenderingSupport()->abandonCount(webPrerender));
EXPECT_EQ(2u, prerenderingSupport()->totalCount());
- // Check that the prerender does not emit an extra cancel when garbage-collecting everything.
+ // Check that the prerender does not emit an extra cancel when
+ // garbage-collecting everything.
close();
EXPECT_EQ(2u, prerenderingSupport()->totalCount());
@@ -401,12 +402,14 @@ TEST_F(PrerenderingTest, FastRemoveElement) {
EXPECT_EQ(1u, prerenderingSupport()->addCount(webPrerender));
EXPECT_EQ(1u, prerenderingSupport()->totalCount());
- // Race removing & starting the prerender against each other, as if the element was removed very quickly.
+ // Race removing & starting the prerender against each other, as if the
+ // element was removed very quickly.
executeScript("removePrerender()");
EXPECT_FALSE(webPrerender.isNull());
webPrerender.didStartPrerender();
- // The page should be totally disconnected from the Prerender at this point, so the console should not have updated.
+ // The page should be totally disconnected from the Prerender at this point,
+ // so the console should not have updated.
EXPECT_EQ(0u, consoleLength());
}
@@ -421,7 +424,8 @@ TEST_F(PrerenderingTest, MutateTarget) {
EXPECT_EQ(0u, prerenderingSupport()->cancelCount(webPrerender));
EXPECT_EQ(1u, prerenderingSupport()->totalCount());
- // Change the href of this prerender, make sure this is treated as a remove and add.
+ // Change the href of this prerender, make sure this is treated as a remove
+ // and add.
executeScript("mutateTarget()");
EXPECT_EQ(1u, prerenderingSupport()->cancelCount(webPrerender));
« no previous file with comments | « third_party/WebKit/Source/web/tests/MHTMLTest.cpp ('k') | third_party/WebKit/Source/web/tests/ProgrammaticScrollTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698