Do not reset NavigationHandle when navigating same-page

content/browser/frame_host/interstitial_page_navigator_impl.cc

Index: content/browser/frame_host/interstitial_page_navigator_impl.cc
diff --git a/content/browser/frame_host/interstitial_page_navigator_impl.cc b/content/browser/frame_host/interstitial_page_navigator_impl.cc
index a19deefa127366eb78f867e56bc8c4c4d7fd73b0..9aec514379798833b3c0a0af4c15126fba5297eb 100644
--- a/content/browser/frame_host/interstitial_page_navigator_impl.cc
+++ b/content/browser/frame_host/interstitial_page_navigator_impl.cc
@@ -45,7 +45,8 @@ void InterstitialPageNavigatorImpl::DidStartProvisionalLoad(
 
 void InterstitialPageNavigatorImpl::DidNavigate(
     RenderFrameHostImpl* render_frame_host,
-    const FrameHostMsg_DidCommitProvisionalLoad_Params& input_params) {
+    const FrameHostMsg_DidCommitProvisionalLoad_Params& input_params,
+    std::unique_ptr<NavigationHandleImpl> navigation_handle) {
   if (navigation_handle_) {
     navigation_handle_->DidCommitNavigation(input_params, false,

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

This looks incorrect now.  We should assume that same-page navigations are
possible on interstitial pages, since other embedders define their own
interstitials.

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

I'm now checking if the navigation that just committed is not same-page before
resetting the NavigationHandle for the interstitial page.

                                             render_frame_host);