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

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

Issue 10386223: Renaming poorly named classes and adding missing unit tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Undo rename of Dispatcher Created 8 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
« no previous file with comments | « chrome/browser/chrome_content_browser_client.cc ('k') | 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 85fb399878272df69d5ea2ca8c84b5e5aa46ad67..34be88e71aef25537a68727ed8dd50d7779f656b 100644
--- a/chrome/browser/prerender/prerender_browsertest.cc
+++ b/chrome/browser/prerender/prerender_browsertest.cc
@@ -126,14 +126,14 @@ class TestPrerenderContents : public PrerenderContents {
public:
TestPrerenderContents(
PrerenderManager* prerender_manager,
- PrerenderTracker* prerender_tracker,
+ PrerenderRenderViewTracker* prerender_render_view_tracker,
Profile* profile,
const GURL& url,
const content::Referrer& referrer,
int expected_number_of_loads,
FinalStatus expected_final_status,
bool prerender_should_wait_for_ready_title)
- : PrerenderContents(prerender_manager, prerender_tracker,
+ : PrerenderContents(prerender_manager, prerender_render_view_tracker,
profile, url, referrer, ORIGIN_LINK_REL_PRERENDER,
PrerenderManager::kNoExperiment),
number_of_loads_(0),
@@ -357,7 +357,7 @@ class WaitForLoadPrerenderContentsFactory : public PrerenderContents::Factory {
virtual PrerenderContents* CreatePrerenderContents(
PrerenderManager* prerender_manager,
- PrerenderTracker* prerender_tracker,
+ PrerenderRenderViewTracker* prerender_render_view_tracker,
Profile* profile,
const GURL& url,
const content::Referrer& referrer,
@@ -371,7 +371,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, prerender_tracker,
+ return new TestPrerenderContents(prerender_manager,
+ prerender_render_view_tracker,
profile, url,
referrer, expected_number_of_loads_,
expected_final_status,
« no previous file with comments | « chrome/browser/chrome_content_browser_client.cc ('k') | chrome/browser/prerender/prerender_contents.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698