Make a separate histogram for MatchComplete Final Status'es and

chrome/browser/prerender/prerender_contents.h

Index: chrome/browser/prerender/prerender_contents.h
===================================================================
--- chrome/browser/prerender/prerender_contents.h	(revision 118385)
+++ chrome/browser/prerender/prerender_contents.h	(working copy)
@@ -81,6 +81,23 @@
   };
   typedef std::list<PendingPrerenderData> PendingPrerenderList;
 
+  // Indicates how this PrerenderContents relates to MatchComplete.
+  // This is important to figure out in what histograms to record the
+  // FinalStatus in, as described below.
+  enum MatchCompleteStatus {
+    // A regular prerender which will be recorded both in Match and
+    // MatchComplete.
+    MC_DEFAULT,
+    // A prerender that used to be a regular prerender, but has since
+    // been replaced by a MachComplete dummy.  Therefore, we will record

[dominich, 2012/01/23 22:41:40]

nit: MatchComplete

[tburkard, 2012/01/24 00:59:56]

Done.

+    // this only for Match, but not for MatchComplete.
+    MC_REPLACED,
+    // A prerender that is a MatchComplete dummy replacing a regular
+    // prerender.  Therefore, we iwll record this only for MatchComplete,

[dominich, 2012/01/23 22:41:40]

nit: will

[tburkard, 2012/01/24 00:59:56]

Done.

+    // but not Match.
+    MC_REPLACEMENT
+  };
+
   virtual ~PrerenderContents();
 
   bool Init();
@@ -106,6 +123,8 @@
   const content::Referrer& referrer() const { return referrer_; }
   bool has_stopped_loading() const { return has_stopped_loading_; }
   bool prerendering_has_started() const { return prerendering_has_started_; }
+  MatchCompleteStatus mc_status() const { return mc_status_; }

[dominich, 2012/01/23 22:41:40]

I understand this is used frequently, but it's Chromium style to not use
abbreviations in variable names. Please expand it to match_complete_status_

[tburkard, 2012/01/24 00:59:56]

Done.

+  void set_mc_status(MatchCompleteStatus status) { mc_status_ = status; }
 
   // Sets the parameter to the value of the associated RenderViewHost's child id
   // and returns a boolean indicating the validity of that id.
@@ -269,6 +288,11 @@
 
   bool prerendering_has_started_;
 
+  // The MatchComplete status of the prerender, indicating how it relates
+  // to being a MatchComplete dummy (see definition of MatchCompleteStatus
+  // above).
+  MatchCompleteStatus mc_status_;
+
   // Tracks whether or not prerendering has been cancelled by calling Destroy.
   // Used solely to prevent double deletion.
   bool prerendering_has_been_cancelled_;