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

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

Issue 11348357: Add observer interface to PrerenderContents. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: final review remediation, clear to land version Created 8 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 | « no previous file | chrome/browser/prerender/prerender_contents.h » ('j') | 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 c93c440e9c102e426db469a1f375241930f8caea..97f7be961289c398b93fe7edfa6ae2875b05b596 100644
--- a/chrome/browser/prerender/prerender_browsertest.cc
+++ b/chrome/browser/prerender/prerender_browsertest.cc
@@ -208,7 +208,6 @@ class TestPrerenderContents : public PrerenderContents {
public:
TestPrerenderContents(
PrerenderManager* prerender_manager,
- PrerenderTracker* prerender_tracker,
Profile* profile,
const GURL& url,
const content::Referrer& referrer,
@@ -216,7 +215,7 @@ class TestPrerenderContents : public PrerenderContents {
int expected_number_of_loads,
FinalStatus expected_final_status,
bool prerender_should_wait_for_ready_title)
- : PrerenderContents(prerender_manager, prerender_tracker, profile, url,
+ : PrerenderContents(prerender_manager, profile, url,
referrer, origin, PrerenderManager::kNoExperiment),
number_of_loads_(0),
expected_number_of_loads_(expected_number_of_loads),
@@ -440,7 +439,6 @@ class WaitForLoadPrerenderContentsFactory : public PrerenderContents::Factory {
virtual PrerenderContents* CreatePrerenderContents(
PrerenderManager* prerender_manager,
- PrerenderTracker* prerender_tracker,
Profile* profile,
const GURL& url,
const content::Referrer& referrer,
@@ -454,7 +452,7 @@ class WaitForLoadPrerenderContentsFactory : public PrerenderContents::Factory {
VLOG(1) << "Creating prerender contents for " << url.path() <<
" with expected final status " << expected_final_status;
VLOG(1) << expected_final_status_queue_.size() << " left in the queue.";
- return new TestPrerenderContents(prerender_manager, prerender_tracker,
+ return new TestPrerenderContents(prerender_manager,
profile, url, referrer, origin,
expected_number_of_loads_,
expected_final_status,
« no previous file with comments | « no previous file | chrome/browser/prerender/prerender_contents.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698