chrome/browser/prerender/prerender_manager_unittest.cc - Issue 8801012: Add support for content::Referrer to prerender

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

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: updates 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

« no previous file with comments | « chrome/browser/prerender/prerender_manager.cc ('k') | chrome/browser/renderer_host/chrome_resource_dispatcher_host_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

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..9a9b6be404a99a97aeaa7309e479a71bbfb6e4a6 100644

--- a/chrome/browser/prerender/prerender_manager_unittest.cc

+++ b/chrome/browser/prerender/prerender_manager_unittest.cc

@@ -27,7 +27,8 @@ class DummyPrerenderContents : public PrerenderContents {

const GURL& url,

FinalStatus expected_final_status)

: PrerenderContents(prerender_manager, prerender_tracker,

- NULL, url, GURL(), ORIGIN_LINK_REL_PRERENDER,

+ NULL, url, content::Referrer(),

+ ORIGIN_LINK_REL_PRERENDER,

PrerenderManager::kNoExperiment),

has_started_(false),

expected_final_status_(expected_final_status) {

@@ -128,7 +129,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 +158,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 +438,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 +469,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