Make a separate histogram for MatchComplete Final Status'es and

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 63 matching lines @@
     PendingPrerenderData(Origin origin,
                          const GURL& url,
                          const content::Referrer& referrer);
 
     Origin origin;
     GURL url;
     content::Referrer referrer;
   };
   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();
 
   static Factory* CreateFactory();
 
   // |source_render_view_host| is the RenderViewHost that initiated
   // prerendering.
   virtual void StartPrerendering(
       const RenderViewHost* source_render_view_host,
       SessionStorageNamespace* session_storage_namespace);
 
   // Verifies that the prerendering is not using too many resources, and kills
   // it if not.
   void DestroyWhenUsingTooManyResources();
 
   RenderViewHost* render_view_host_mutable();
   const RenderViewHost* render_view_host() const;
   string16 title() const { return title_; }
   int32 page_id() const { return page_id_; }
   GURL icon_url() const { return icon_url_; }
   const GURL& prerender_url() const { return prerender_url_; }
   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.
   virtual bool GetChildId(int* child_id) const;
 
   // Sets the parameter to the value of the associated RenderViewHost's route id
   // and returns a boolean indicating the validity of that id.
   virtual bool GetRouteId(int* route_id) const;
 
   // Set the final status for how the PrerenderContents was used. This
@@ skipping 143 matching lines @@
   std::vector<GURL> alias_urls_;
 
   bool has_stopped_loading_;
 
   // This must be the same value as the PrerenderTracker has recorded for
   // |this|, when |this| has a RenderView.
   FinalStatus final_status_;
 
   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_;
 
   // Time at which we started to load the URL.  This is used to compute
   // the time elapsed from initiating a prerender until the time the
   // (potentially only partially) prerendered page is shown to the user.
   base::TimeTicks load_start_time_;
 
   // Process Metrics of the render process associated with the
@@ skipping 19 matching lines @@
 
   // List of all pages the prerendered page has tried to prerender.
   PendingPrerenderList pending_prerender_list_;
 
   DISALLOW_COPY_AND_ASSIGN(PrerenderContents);
 };
 
 }  // namespace prerender
 
 #endif  // CHROME_BROWSER_PRERENDER_PRERENDER_CONTENTS_H_