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

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

Issue 6966017: Remove a chrome dependency by removing Prerender from ResourceDispatcherHost. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Another round of comments Created 9 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
Index: chrome/browser/prerender/prerender_browsertest.cc
diff --git a/chrome/browser/prerender/prerender_browsertest.cc b/chrome/browser/prerender/prerender_browsertest.cc
index 3f166476b07cdafb6e924ec0d1b809cd9419da84..58e26082d8d7b80896d631cf72db4bd71319e8d4 100644
--- a/chrome/browser/prerender/prerender_browsertest.cc
+++ b/chrome/browser/prerender/prerender_browsertest.cc
@@ -71,11 +71,13 @@ class TestPrerenderContents : public PrerenderContents {
TestPrerenderContents(
PrerenderManager* prerender_manager,
Profile* profile,
mmenke 2011/05/24 18:38:09 This argument order is kind of funky. I'd go with
dominich 2011/05/24 19:45:41 Done.
+ PrerenderTracker* prerender_tracker,
const GURL& url,
const GURL& referrer,
int number_of_loads,
FinalStatus expected_final_status)
- : PrerenderContents(prerender_manager, profile, url, referrer),
+ : PrerenderContents(prerender_manager, prerender_tracker, profile,
+ url, referrer),
number_of_loads_(0),
expected_number_of_loads_(number_of_loads),
expected_final_status_(expected_final_status),
@@ -203,6 +205,7 @@ class WaitForLoadPrerenderContentsFactory : public PrerenderContents::Factory {
virtual PrerenderContents* CreatePrerenderContents(
PrerenderManager* prerender_manager,
+ PrerenderTracker* prerender_tracker,
Profile* profile,
const GURL& url,
const GURL& referrer) OVERRIDE {
@@ -214,7 +217,8 @@ 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, profile, url,
+ return new TestPrerenderContents(prerender_manager, profile,
+ prerender_tracker, url,
referrer, number_of_loads_,
expected_final_status);
}

Powered by Google App Engine
This is Rietveld 408576698