Prerender: Remove MatchComplete status.

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_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 73 matching lines @@
     // Signals that the prerender has stopped running. A PrerenderContents with
     // an unset final status will always call OnPrerenderStop before being
     // destroyed.
     virtual void OnPrerenderStop(PrerenderContents* contents) = 0;
 
    protected:
     Observer();
     virtual ~Observer() = 0;
   };
 
-  // Indicates how this PrerenderContents relates to MatchComplete. This is to
-  // figure out which histograms to use to record the FinalStatus, Match (record
-  // all prerenders and control group prerenders) or MatchComplete (record
-  // running prerenders only in the way they would have been recorded in the
-  // control group).
-  // TODO(pasko): Remove the dead code associated with non-default
-  // MatchCompleteStatus.
-  enum MatchCompleteStatus {
-    // A regular prerender which will be recorded both in Match and
-    // MatchComplete.
-    MATCH_COMPLETE_DEFAULT,
-    // A prerender that used to be a regular prerender, but has since been
-    // replaced by a MatchComplete dummy. Therefore, we will record this only
-    // for Match, but not for MatchComplete.
-    MATCH_COMPLETE_REPLACED,
-    // A prerender that is a MatchComplete dummy replacing a regular prerender.
-    // In the control group, our prerender never would have been canceled, so
-    // we record in MatchComplete but not Match.
-    MATCH_COMPLETE_REPLACEMENT,
-    // A prerender that is a MatchComplete dummy, early in the process of being
-    // created. This prerender should not fail. Record for MatchComplete, but
-    // not Match.
-    MATCH_COMPLETE_REPLACEMENT_PENDING,
-  };
-
   ~PrerenderContents() override;
 
   // All observers of a PrerenderContents are removed after the OnPrerenderStop
   // event is sent, so there is no need to call RemoveObserver() in the normal
   // use case.
   void AddObserver(Observer* observer);
   void RemoveObserver(Observer* observer);
 
-  // For MatchComplete correctness, create a dummy replacement prerender
-  // contents to stand in for this prerender contents that (which we are about
-  // to destroy).
-  PrerenderContents* CreateMatchCompleteReplacement();
-
   bool Init();
 
   static Factory* CreateFactory();
 
   // Returns a PrerenderContents from the given web_contents, if it's used for
   // prerendering. Otherwise returns NULL. Handles a NULL input for
   // convenience.
   static PrerenderContents* FromWebContents(content::WebContents* web_contents);
 
   // Start rendering the contents in the prerendered state. If
@@ skipping 14 matching lines @@
   const content::RenderViewHost* GetRenderViewHost() const;
 
   PrerenderManager* prerender_manager() { return prerender_manager_; }
 
   base::string16 title() const { return title_; }
   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 has_finished_loading() const { return has_finished_loading_; }
   bool prerendering_has_started() const { return prerendering_has_started_; }
-  MatchCompleteStatus match_complete_status() const {
-    return match_complete_status_;
-  }
-  void set_match_complete_status(MatchCompleteStatus status) {
-    match_complete_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;
 
   FinalStatus final_status() const { return final_status_; }
@@ skipping 173 matching lines @@
   // 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_;
 
   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_;
 
   // Process Metrics of the render process associated with the
   // RenderViewHost for this object.
   std::unique_ptr<base::ProcessMetrics> process_metrics_;
 
   std::unique_ptr<WebContentsDelegateImpl> web_contents_delegate_;
 
@@ skipping 19 matching lines @@
   // A running tally of the number of bytes this prerender has caused to be
   // transferred over the network for resources.  Updated with AddNetworkBytes.
   int64_t network_bytes_;
 
   DISALLOW_COPY_AND_ASSIGN(PrerenderContents);
 };
 
 }  // namespace prerender
 
 #endif  // CHROME_BROWSER_PRERENDER_PRERENDER_CONTENTS_H_