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

Unified Diff: chrome/browser/prerender/prerender_browsertest.cc

Issue 10257003: Enable Prerender API browsertests (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase to trunk Created 8 years, 7 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_browsertest.cc
diff --git a/chrome/browser/prerender/prerender_browsertest.cc b/chrome/browser/prerender/prerender_browsertest.cc
index cb9b6608176df2d552efb3a9d6ff3dd27811e365..0c6a9bd01940e49dc28458cb3a0a05ce0adda4a7 100644
--- a/chrome/browser/prerender/prerender_browsertest.cc
+++ b/chrome/browser/prerender/prerender_browsertest.cc
@@ -985,10 +985,8 @@ IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest, PrerenderPage) {
ASSERT_TRUE(IsEmptyPrerenderLinkManager());
}
-// TODO(gavinp): After https://bugs.webkit.org/show_bug.cgi?id=85005 lands,
-// enable this test.
IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest,
dominich 2012/05/15 15:52:04 nit: this can be one line i think
gavinp 2012/05/18 02:59:52 Done.
- DISABLED_PrerenderPageRemovingLink) {
+ PrerenderPageRemovingLink) {
set_loader_path("files/prerender/prerender_loader_removing_links.html");
set_loader_query_and_fragment("?links_to_insert=1&links_to_remove=1");
PrerenderTestURL("files/prerender/prerender_page.html",
@@ -997,10 +995,8 @@ IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest,
ASSERT_TRUE(IsEmptyPrerenderLinkManager());
}
-// TODO(gavinp): After https://bugs.webkit.org/show_bug.cgi?id=85005 lands,
-// enable this test.
IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest,
- DISABLED_PrerenderPageRemovingLinkWithTwoLinks) {
+ PrerenderPageRemovingLinkWithTwoLinks) {
set_loader_path("files/prerender/prerender_loader_removing_links.html");
set_loader_query_and_fragment("?links_to_insert=2&links_to_remove=2");
PrerenderTestURL("files/prerender/prerender_page.html",
@@ -1009,11 +1005,9 @@ IN_PROC_BROWSER_TEST_F(PrerenderBrowserTest,
ASSERT_TRUE(IsEmptyPrerenderLinkManager());
}
-// TODO(gavinp): After https://bugs.webkit.org/show_bug.cgi?id=85005 lands,
-// enable this test.
IN_PROC_BROWSER_TEST_F(
dominich 2012/05/15 15:52:04 nit: bring PrerenderBrowserTest up a line
gavinp 2012/05/18 02:59:52 Done.
PrerenderBrowserTest,
- DISABLED_PrerenderPageRemovingLinkWithTwoLinksRemovingOne) {
+ PrerenderPageRemovingLinkWithTwoLinksRemovingOne) {
dominich 2012/05/15 15:52:04 add a test with insert=1 and remove=2
gavinp 2012/05/18 02:59:52 Done.
set_loader_path("files/prerender/prerender_loader_removing_links.html");
set_loader_query_and_fragment("?links_to_insert=2&links_to_remove=1");
PrerenderTestURL("files/prerender/prerender_page.html",
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698