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

Unified Diff: chrome/browser/prerender/prerender_contents.h

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/prerender/prerender_browsertest.cc ('k') | chrome/browser/prerender/prerender_contents.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/prerender/prerender_contents.h
diff --git a/chrome/browser/prerender/prerender_contents.h b/chrome/browser/prerender/prerender_contents.h
index 9546da7eac65d8f26a83ac92bc72ee37c7317374..2f8f0ae32ddff7005e644681ba62d62b03de829b 100644
--- a/chrome/browser/prerender/prerender_contents.h
+++ b/chrome/browser/prerender/prerender_contents.h
@@ -40,7 +40,7 @@ namespace prerender {
class PrerenderManager;
class PrerenderRenderViewHostObserver;
-class PrerenderTracker;
+class PrerenderRenderViewTracker;
class PrerenderContents : public content::NotificationObserver,
public content::WebContentsObserver {
@@ -53,10 +53,10 @@ class PrerenderContents : public content::NotificationObserver,
virtual ~Factory() {}
// Ownership is not transfered through this interface as prerender_manager,
- // prerender_tracker, and profile are stored as weak pointers.
+ // prerender_render_view_tracker, and profile are stored as weak pointers.
virtual PrerenderContents* CreatePrerenderContents(
PrerenderManager* prerender_manager,
- PrerenderTracker* prerender_tracker,
+ PrerenderRenderViewTracker* prerender_render_view_tracker,
Profile* profile,
const GURL& url,
const content::Referrer& referrer,
@@ -216,7 +216,7 @@ class PrerenderContents : public content::NotificationObserver,
protected:
PrerenderContents(PrerenderManager* prerender_manager,
- PrerenderTracker* prerender_tracker,
+ PrerenderRenderViewTracker* prerender_render_view_tracker,
Profile* profile,
const GURL& url,
const content::Referrer& referrer,
@@ -266,7 +266,7 @@ class PrerenderContents : public content::NotificationObserver,
PrerenderManager* prerender_manager_;
// The prerender tracker tracking prerenders.
- PrerenderTracker* prerender_tracker_;
+ PrerenderRenderViewTracker* prerender_render_view_tracker_;
// The URL being prerendered.
GURL prerender_url_;
@@ -298,8 +298,8 @@ class PrerenderContents : public content::NotificationObserver,
// True when the main frame has finished loading.
bool has_finished_loading_;
- // This must be the same value as the PrerenderTracker has recorded for
- // |this|, when |this| has a RenderView.
+ // This must be the same value as the PrerenderRenderViewTracker has recorded
+ // for |this|, when |this| has a RenderView.
FinalStatus final_status_;
// The MatchComplete status of the prerender, indicating how it relates
« no previous file with comments | « chrome/browser/prerender/prerender_browsertest.cc ('k') | chrome/browser/prerender/prerender_contents.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698