Renaming poorly named classes and adding missing unit tests.

chrome/browser/prerender/prerender_contents.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_PRERENDER_PRERENDER_CONTENTS_H_
 #define CHROME_BROWSER_PRERENDER_PRERENDER_CONTENTS_H_
 #pragma once
 
 #include <list>
 #include <string>
@@ skipping 22 matching lines @@
 class RenderViewHost;
 class RenderViewHostDelegate;
 class SessionStorageNamespace;
 class WebContents;
 }
 
 namespace prerender {
 
 class PrerenderManager;
 class PrerenderRenderViewHostObserver;
-class PrerenderTracker;
+class PrerenderRenderViewTracker;
 
 class PrerenderContents : public content::NotificationObserver,
                           public content::WebContentsObserver {
  public:
   // PrerenderContents::Create uses the currently registered Factory to create
   // the PrerenderContents. Factory is intended for testing.
   class Factory {
    public:
     Factory() {}
     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,
         Origin origin,
         uint8 experiment_id) = 0;
 
    private:
     DISALLOW_COPY_AND_ASSIGN(Factory);
   };
 
@@ skipping 139 matching lines @@
 
   // Returns true if |url| corresponds to a pending prerender.
   bool IsPendingEntry(const GURL& url) const;
 
   // Reissues any pending prerender requests from the prerendered page.  Also
   // clears the list of pending requests.
   void StartPendingPrerenders();
 
  protected:
   PrerenderContents(PrerenderManager* prerender_manager,
-                    PrerenderTracker* prerender_tracker,
+                    PrerenderRenderViewTracker* prerender_render_view_tracker,
                     Profile* profile,
                     const GURL& url,
                     const content::Referrer& referrer,
                     Origin origin,
                     uint8 experiment_id);
 
   // Called whenever a RenderViewHost is created for prerendering.  Only called
   // once the RenderViewHost has a RenderView and RenderWidgetHostView.
   virtual void OnRenderViewHostCreated(
       content::RenderViewHost* new_render_view_host);
@@ skipping 29 matching lines @@
   // Returns the RenderViewHost Delegate for this prerender.
   content::WebContents* GetWebContents();
 
   // Returns the ProcessMetrics for the render process, if it exists.
   base::ProcessMetrics* MaybeGetProcessMetrics();
 
   // The prerender manager owning this object.
   PrerenderManager* prerender_manager_;
 
   // The prerender tracker tracking prerenders.
-  PrerenderTracker* prerender_tracker_;
+  PrerenderRenderViewTracker* prerender_render_view_tracker_;
 
   // The URL being prerendered.
   GURL prerender_url_;
 
   // The referrer.
   content::Referrer referrer_;
 
   // The profile being used
   Profile* profile_;
 
@@ skipping 11 matching lines @@
   // A vector of URLs that this prerendered page matches against.
   // This array can contain more than element as a result of redirects,
   // such as HTTP redirects or javascript redirects.
   std::vector<GURL> alias_urls_;
 
   bool has_stopped_loading_;
 
   // 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
   // to being a MatchComplete dummy (see definition of MatchCompleteStatus
   // above).
   MatchCompleteStatus match_complete_status_;
 
   // Tracks whether or not prerendering has been cancelled by calling Destroy.
   // Used solely to prevent double deletion.
   bool prerendering_has_been_cancelled_;
@@ skipping 32 matching lines @@
 
   // The size of the WebView from the launching page.
   gfx::Size size_;
 
   DISALLOW_COPY_AND_ASSIGN(PrerenderContents);
 };
 
 }  // namespace prerender
 
 #endif  // CHROME_BROWSER_PRERENDER_PRERENDER_CONTENTS_H_