Do not reset NavigationHandle when navigating same-page

content/browser/frame_host/navigator.h

Index: content/browser/frame_host/navigator.h
diff --git a/content/browser/frame_host/navigator.h b/content/browser/frame_host/navigator.h
index 3424428fc070d950bbe90ae192d2ce6fdfb12075..f1e87c2eb65e32a9a3ca2410c0e51bea64b600fb 100644
--- a/content/browser/frame_host/navigator.h
+++ b/content/browser/frame_host/navigator.h
@@ -75,7 +75,8 @@ class CONTENT_EXPORT Navigator : public base::RefCounted<Navigator> {
   // The RenderFrameHostImpl has committed a navigation.

[Charlie Reis, 2016/11/03 22:02:22]

nit: Mention something about how we shouldn't assume that |navigation_handle|
matches RFH's current NavigationHandle, and that DidNavigate is responsible for
resetting |navigation_handle| at the end of this method (and shouldn't keep it
alive).

[clamy, 2016/11/04 14:18:21]

Done.

   virtual void DidNavigate(
       RenderFrameHostImpl* render_frame_host,
-      const FrameHostMsg_DidCommitProvisionalLoad_Params& params) {}
+      const FrameHostMsg_DidCommitProvisionalLoad_Params& params,
+      std::unique_ptr<NavigationHandleImpl> navigation_handle) {}
 
   // Called by the NavigationController to cause the Navigator to navigate
   // to the current pending entry. The NavigationController should be called