Accessing navigation information via webcontents

chrome/browser/net/resource_prefetch_predictor_observer.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_NET_RESOURCE_PREFETCH_PREDICTOR_OBSERVER_H_
 #define CHROME_BROWSER_NET_RESOURCE_PREFETCH_PREDICTOR_OBSERVER_H_
 
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/predictors/resource_prefetch_predictor.h"
+#include "content/public/browser/resource_request_info.h"
 #include "content/public/common/resource_type.h"
 
 namespace net {
 class URLRequest;
 }
 
 class GURL;
 
 namespace chrome_browser_net {
 
 // Observes resource requests in the ResourceDispatcherHostDelegate and notifies
 // the ResourcePrefetchPredictor about the ones it is interested in.
 //  - Has an instance per profile, and is owned by the corresponding
 //    ProfileIOData.
 //  - Needs to be constructed on UI thread. Rest of the functions can only be
 //    called on the IO thread. Can be destroyed on UI or IO thread.

[alexilin, 2016/12/05 13:20:00]

Probably we should update this comment. I'm not sure whether class comment
describes public interface only (lizeb?).

 class ResourcePrefetchPredictorObserver {
+

[ahemery, 2016/12/05 09:38:45]

Not sure this is okay in a header file, even if inside a namespace

[alexilin, 2016/12/05 13:20:00]

We generally avoid the use of using declaration inside header files.
Please use fully qualified name here.

+  using URLRequestSummary =

[ahemery, 2016/12/06 14:31:11]

Removed and full names used in header file

+      predictors::ResourcePrefetchPredictor::URLRequestSummary;
+
  public:
   explicit ResourcePrefetchPredictorObserver(
       predictors::ResourcePrefetchPredictor* predictor);
   ~ResourcePrefetchPredictorObserver();
 
   // Parts of the ResourceDispatcherHostDelegate that we want to observe.
-  void OnRequestStarted(net::URLRequest* request,
-                        content::ResourceType resource_type,
-                        int child_id,
-                        int frame_id);
-  void OnRequestRedirected(const GURL& redirect_url, net::URLRequest* request);
-  void OnResponseStarted(net::URLRequest* request);
+  void OnRequestStarted(
+      net::URLRequest* request,
+      content::ResourceType resource_type,
+      const content::ResourceRequestInfo::WebContentsGetter&
+          web_contents_getter);
+  void OnRequestRedirected(
+      net::URLRequest* request,
+      const GURL& redirect_url,
+      const content::ResourceRequestInfo::WebContentsGetter&
+          web_contents_getter);
+  void OnResponseStarted(
+      net::URLRequest* request,
+      const content::ResourceRequestInfo::WebContentsGetter&
+          web_contents_getter);
 
  private:
+  void OnRequestStartedOnUIThread(
+    std::unique_ptr<URLRequestSummary> summary,
+    const content::ResourceRequestInfo::WebContentsGetter&
+        web_contents_getter) const;
+  void OnRequestRedirectedOnUIThread(
+    std::unique_ptr<URLRequestSummary> summary,
+    const content::ResourceRequestInfo::WebContentsGetter&
+        web_contents_getter) const;
+  void OnResponseStartedOnUIThread(
+    std::unique_ptr<URLRequestSummary> summary,
+    const content::ResourceRequestInfo::WebContentsGetter&
+        web_contents_getter) const;
+
+  void RetrieveNavigationID(
+    predictors::NavigationID& navigation_id,
+    const content::ResourceRequestInfo::WebContentsGetter&
+        web_contents_getter) const;
+
   // Owned by profile.
   base::WeakPtr<predictors::ResourcePrefetchPredictor> predictor_;
 
   DISALLOW_COPY_AND_ASSIGN(ResourcePrefetchPredictorObserver);
 };
 
 }  // namespace chrome_browser_net
 
 #endif  // CHROME_BROWSER_NET_RESOURCE_PREFETCH_PREDICTOR_OBSERVER_H_