Add methods to NavigationHandle to allow refactoring webNavigation to use it.

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 6248b18accd8b00aeed381575909ca6fac07f16f..90d28f48e959aa3c6d2d258becb7fcd450899549 100644
--- a/content/browser/frame_host/navigation_handle_impl.h
+++ b/content/browser/frame_host/navigation_handle_impl.h
@@ -70,12 +70,20 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
   static scoped_ptr<NavigationHandleImpl> Create(
       const GURL& url,
       FrameTreeNode* frame_tree_node,
+      bool is_synchronous,
+      bool is_srcdoc,
       const base::TimeTicks& navigation_start);
   ~NavigationHandleImpl() override;
 
   // NavigationHandle implementation:
   const GURL& GetURL() override;
   bool IsInMainFrame() override;
+  bool IsParentMainFrame() override;
+  bool IsSynchronousNavigation() override;
+  bool IsSrcdoc() override;
+  bool WasServerRedirect() override;
+  int GetFrameTreeNodeId() override;
+  int GetParentFrameTreeNodeId() override;
   const base::TimeTicks& NavigationStart() override;
   bool IsPost() override;
   const Referrer& GetReferrer() override;
@@ -201,6 +209,8 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
 
   NavigationHandleImpl(const GURL& url,
                        FrameTreeNode* frame_tree_node,
+                       bool is_synchronous,
+                       bool is_srcdoc,
                        const base::TimeTicks& navigation_start);
 
   NavigationThrottle::ThrottleCheckResult CheckWillStartRequest();
@@ -223,6 +233,9 @@ class CONTENT_EXPORT NavigationHandleImpl : public NavigationHandle {
   net::Error net_error_code_;
   RenderFrameHostImpl* render_frame_host_;
   bool is_same_page_;
+  bool is_synchronous_;

[clamy, 2016/02/08 12:54:58]

Can is_synchronous_ and is_srcdoc_ be made const? I don't think they are
supposed to change over the course of the navigation.

[nasko, 2016/02/08 16:58:55]

Done.

+  bool is_srcdoc_;
+  bool was_redirected_;
   scoped_refptr<net::HttpResponseHeaders> response_headers_;
 
   // The state the navigation is in.