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

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

Issue 2697513009: Enable forced prerenders on wifi (Closed)
Patch Set: Created 3 years, 10 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 | « chrome/browser/prerender/prerender_origin.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_unittest.cc
diff --git a/chrome/browser/prerender/prerender_unittest.cc b/chrome/browser/prerender/prerender_unittest.cc
index 45df72b8bb9a90e7d6e15712d3a3aafc8a48042b..be3bfa0e35b69dbc139614fb9a7b89b77756182d 100644
--- a/chrome/browser/prerender/prerender_unittest.cc
+++ b/chrome/browser/prerender/prerender_unittest.cc
@@ -1341,16 +1341,16 @@ TEST_F(PrerenderTest, PrerenderSilenceAllowsForcedCellular) {
GURL url("http://www.google.com/");
DummyPrerenderContents* prerender_contents =
prerender_manager()->CreateNextPrerenderContents(
- url, ORIGIN_EXTERNAL_REQUEST_FORCED_CELLULAR,
+ url, ORIGIN_EXTERNAL_REQUEST_FORCED_PRERENDER,
FINAL_STATUS_MANAGER_SHUTDOWN);
std::unique_ptr<PrerenderHandle> prerender_handle =
- prerender_manager()->AddPrerenderOnCellularFromExternalRequest(
+ prerender_manager()->AddForcedPrerenderFromExternalRequest(
url, content::Referrer(), nullptr, gfx::Rect(kSize));
EXPECT_TRUE(prerender_handle);
EXPECT_TRUE(prerender_handle->IsPrerendering());
EXPECT_TRUE(prerender_contents->prerendering_has_started());
EXPECT_EQ(prerender_contents, prerender_handle->contents());
- EXPECT_EQ(ORIGIN_EXTERNAL_REQUEST_FORCED_CELLULAR,
+ EXPECT_EQ(ORIGIN_EXTERNAL_REQUEST_FORCED_PRERENDER,
prerender_handle->contents()->origin());
}
@@ -1406,8 +1406,7 @@ TEST_F(PrerenderTest, PrerenderSilenceAllowsAfterExpiration) {
TEST_F(PrerenderTest, PrerenderAllowedForOfflineAndForcedCellular) {
const Origin origins[] = {
- ORIGIN_EXTERNAL_REQUEST_FORCED_CELLULAR,
- ORIGIN_OFFLINE,
+ ORIGIN_EXTERNAL_REQUEST_FORCED_PRERENDER, ORIGIN_OFFLINE,
};
EnablePrerender();
@@ -1428,7 +1427,7 @@ TEST_F(PrerenderTest, PrerenderAllowedForOfflineAndForcedCellular) {
prerender_contents = prerender_manager()->CreateNextPrerenderContents(
url, origin, FINAL_STATUS_USED);
prerender_handle =
- prerender_manager()->AddPrerenderOnCellularFromExternalRequest(
+ prerender_manager()->AddForcedPrerenderFromExternalRequest(
url, content::Referrer(), nullptr, gfx::Rect(kSize));
}
EXPECT_TRUE(prerender_handle);
« no previous file with comments | « chrome/browser/prerender/prerender_origin.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698