predictors: Several improvements for redirects database.

chrome/browser/predictors/resource_prefetch_predictor.h

 // Copyright 2014 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_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_
 #define CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_
 
 #include <stddef.h>
 
 #include <map>
@@ skipping 280 matching lines @@
   static bool IsHandledSubresource(net::URLRequest* request,
                                    content::ResourceType resource_type);
 
   // Returns true if the subresource has a supported type.
   static bool IsHandledResourceType(content::ResourceType resource_type,
                                     const std::string& mime_type);
 
   // Returns true if the request (should have a response in it) is "no-store".
   static bool IsNoStore(const net::URLRequest* request);
 
-  // Returns true iff |redirect_data_map| contains confident redirect endpoint
-  // for |entry_point| and assigns it to the |redirect_endpoint|.
-  static bool GetRedirectEndpoint(const std::string& entry_point,
-                                  const RedirectDataMap& redirect_data_map,
-                                  std::string* redirect_endpoint);
-
   static void SetAllowPortInUrlsForTesting(bool state);
 
   // KeyedService methods override.
   void Shutdown() override;
 
   // Functions called on different network events pertaining to the loading of
   // main frame resource or sub resources.
   void OnMainFrameRequest(const URLRequestSummary& request);
   void OnMainFrameResponse(const URLRequestSummary& response);
   void OnMainFrameRedirect(const URLRequestSummary& response);
   void OnSubresourceResponse(const URLRequestSummary& response);
 
   // Called when onload completes for a navigation. We treat this point as the
   // "completion" of the navigation. The resources requested by the page up to
   // this point are the only ones considered for prefetching.
   void OnNavigationComplete(const NavigationID& nav_id_without_timing_info);
 
+  // Returns true iff one of the following conditions is true
+  // * |redirect_data_map| contains confident redirect endpoint for
+  //   |entry_point| and assigns it to the |redirect_endpoint|
+  //
+  // * |redirect_data_map| doens't contain an entry for |entry_point| and
+  //   assings |entry_point| to the |redirect_endpoint|.
+  bool GetRedirectEndpoint(const std::string& entry_point,
+                           const RedirectDataMap& redirect_data_map,
+                           std::string* redirect_endpoint) const;
+
   // Returns true iff there is PrefetchData that can be used for a
   // |main_frame_url| and fills |prediction| with resources that need to be
   // prefetched. |prediction| pointer may be equal nullptr to get return value
   // only.
   bool GetPrefetchData(const GURL& main_frame_url,
                        Prediction* prediction) const;
 
   // Returns true iff the |data_map| contains PrefetchData that can be used
   // for a |main_frame_key| and fills |urls| with resources that need to be
   // prefetched. |urls| pointer may be equal nullptr to get return value only.
@@ skipping 131 matching lines @@
 
  private:
   ResourcePrefetchPredictor* predictor_;
 
   DISALLOW_COPY_AND_ASSIGN(TestObserver);
 };
 
 }  // namespace predictors
 
 #endif  // CHROME_BROWSER_PREDICTORS_RESOURCE_PREFETCH_PREDICTOR_H_