Add a NavigationThrottle to the public content/ interface

content/browser/frame_host/navigation_handle_impl.h

Index: content/browser/frame_host/navigation_handle_impl.h
diff --git a/content/browser/frame_host/navigation_handle_impl.h b/content/browser/frame_host/navigation_handle_impl.h
index 33e51ddab278cf281b89fe998d32159acc2210b0..7adab1c0157eb7b36954965192001030b4a14761 100644
--- a/content/browser/frame_host/navigation_handle_impl.h
+++ b/content/browser/frame_host/navigation_handle_impl.h
@@ -9,7 +9,9 @@
 
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/scoped_vector.h"
 #include "content/common/content_export.h"
+#include "content/public/browser/navigation_throttle.h"
 #include "url/gurl.h"
 
 namespace content {
@@ -52,7 +54,10 @@ struct NavigationRequestInfo;
 // the RenderFrameHost still apply.
 class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
  public:
+  // This is passed back to the NavigationResourceThrottle on the network
+  // stack, to let it know whether the navigation should go on or be stopped.

[carlosk, 2015/08/28 16:40:24]

I don't understand how this comments fits here.

[clamy, 2015/09/03 15:30:51]

Because it comes from an earlier version of the patch where the check result
enum was defined here (but forgot to remove the comment apparently :).

   static scoped_ptr<NavigationHandleImpl> Create(const GURL& url,
+                                                 const GURL& validated_url,
                                                  const bool is_main_frame,
                                                  NavigatorDelegate* delegate);
 
@@ -60,10 +65,18 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
 
   // NavigationHandle implementation:
   const GURL& GetURL() const override;
-  net::Error GetNetErrorCode() const override;
+  const GURL& GetValidatedURL() const override;
   bool IsInMainFrame() const override;
+  bool IsPost() const override;
+  const Referrer& GetSanitizedReferrer() const override;
+  bool HasUserGesture() const override;
+  ui::PageTransition GetPageTransition() const override;
+  bool IsExternalProtocol() const override;
+  net::Error GetNetErrorCode() const override;
   bool HasCommittedDocument() const override;
   bool HasCommittedErrorPage() const override;
+  void RegisterThrottle(
+      scoped_ptr<NavigationThrottle> navigation_throttle) override;
 
   void set_net_error_code(net::Error net_error_code) {
     net_error_code_ = net_error_code;
@@ -78,6 +91,22 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
     is_transferring_ = is_transferring;
   }
 
+  // Called when the URLRequest will start in the network stack.
+  NavigationThrottle::ThrottleCheckResult WillStartRequest(
+      bool is_post,
+      const Referrer& sanitized_referrer,
+      bool has_user_gesture,
+      ui::PageTransition transition,
+      bool is_external_protocol);
+
+  // Called when the URLRequest will be redirected in the network stack.
+  NavigationThrottle::ThrottleCheckResult WillRedirectRequest(
+      const GURL& new_url,
+      const GURL& new_validated_url,
+      bool new_method_is_post,
+      const GURL& new_referrer_url,
+      bool new_is_external_protocol);
+
   // Called when the navigation was redirected. This will update the |url_| and
   // inform the delegate.
   void DidRedirectNavigation(const GURL& new_url);
@@ -90,19 +119,30 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
   // Used to track the state the navigation is currently in.
   enum State {
     DID_START = 0,
+    WILL_SEND_REQUEST,
+    WILL_REDIRECT_REQUEST,
     DID_COMMIT,
     DID_COMMIT_ERROR_PAGE,
   };
 
   NavigationHandleImpl(const GURL& url,
+                       const GURL& validated_url,
                        const bool is_main_frame,
                        NavigatorDelegate* delegate);
 
   // See NavigationHandle for a description of those member variables.
   GURL url_;
+  GURL validated_url_;
+  const bool is_main_frame_;
+  bool is_post_;
+  Referrer sanitized_referrer_;
+  bool has_user_gesture_;
+  ui::PageTransition transition_;
+  bool is_external_protocol_;
   net::Error net_error_code_;
+
+  // The state the navigation is in.
   State state_;
-  const bool is_main_frame_;
 
   // Whether the navigation is in the middle of a transfer. Set to false when
   // the DidStartProvisionalLoad is received from the new renderer.
@@ -112,6 +152,9 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
   // navigation.
   NavigatorDelegate* delegate_;
 
+  // A list of Throttles registered for this navigation.
+  ScopedVector<NavigationThrottle> throttles_;
+
   DISALLOW_COPY_AND_ASSIGN(NavigationHandleImpl);
 };