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

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

Issue 8801012: Add support for content::Referrer to prerender (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 9 years 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: chrome/browser/prerender/prerender_manager_unittest.cc
diff --git a/chrome/browser/prerender/prerender_manager_unittest.cc b/chrome/browser/prerender/prerender_manager_unittest.cc
index d835e02826a8ad37954fb704e8ca72ccfec365d4..79dddb15f3c9e0e0c8243579a32af55a50f44d76 100644
--- a/chrome/browser/prerender/prerender_manager_unittest.cc
+++ b/chrome/browser/prerender/prerender_manager_unittest.cc
@@ -128,7 +128,7 @@ class TestPrerenderManager : public PrerenderManager {
bool AddSimplePrerender(const GURL& url) {
return AddPrerenderFromLinkRelPrerender(-1, -1,
url,
- GURL());
+ content::Referrer());
}
void set_rate_limit_enabled(bool enabled) {
@@ -157,7 +157,7 @@ class TestPrerenderManager : public PrerenderManager {
virtual PrerenderContents* CreatePrerenderContents(
const GURL& url,
- const GURL& referrer,
+ const content::Referrer& referrer,
Origin origin,
uint8 experiment_id) OVERRIDE {
DCHECK(next_prerender_contents_.get());
@@ -437,7 +437,7 @@ TEST_F(PrerenderManagerTest, PendingPrerenderTest) {
EXPECT_TRUE(prerender_manager()->AddPrerenderFromLinkRelPrerender(
child_id, route_id,
- pending_url, url));
+ pending_url, content::Referrer(url, WebKit::WebReferrerPolicyDefault)));
EXPECT_TRUE(prerender_manager()->IsPendingEntry(pending_url));
EXPECT_TRUE(prerender_contents->has_started());
@@ -468,7 +468,7 @@ TEST_F(PrerenderManagerTest, SourceRenderViewClosed) {
url,
FINAL_STATUS_MANAGER_SHUTDOWN);
EXPECT_FALSE(prerender_manager()->AddPrerenderFromLinkRelPrerender(
- 100, 100, url, GURL()));
+ 100, 100, url, content::Referrer()));
}
// Tests that the prerender manager ignores fragment references when matching

Powered by Google App Engine
This is Rietveld 408576698