Don't clear pending NavigationParams until didStopLoading.

content/renderer/render_frame_impl.cc

Index: content/renderer/render_frame_impl.cc
diff --git a/content/renderer/render_frame_impl.cc b/content/renderer/render_frame_impl.cc
index 1ababbbe32c6b05884bfe936730cea01f86bd89b..4b99d214b793ebede323b50c77423e6b5ae0172b 100644
--- a/content/renderer/render_frame_impl.cc
+++ b/content/renderer/render_frame_impl.cc
@@ -4799,6 +4799,9 @@ void RenderFrameImpl::didStopLoading() {
                "id", routing_id_);
   render_view_->FrameDidStopLoading(frame_);
   Send(new FrameHostMsg_DidStopLoading(routing_id_));
+
+  // Clear any pending NavigationParams if they didn't get used.
+  pending_navigation_params_.reset();
 }
 
 void RenderFrameImpl::didChangeLoadProgress(double load_progress) {
@@ -5639,9 +5642,6 @@ void RenderFrameImpl::NavigateInternal(
     if (!frame_->isLoading() && !has_history_navigation_in_frame)
       Send(new FrameHostMsg_DidStopLoading(routing_id_));
   }
-
-  // In case LoadRequest failed before didCreateDataSource was called.
-  pending_navigation_params_.reset();

[Avi (use Gerrit), 2016/07/16 02:29:13]

OMG I hate pending params. When can we kill them entirely?

[Charlie Reis, 2016/07/22 18:34:00]

Yes, I would love to see them gone.  I'm guessing it's just a large plumbing job
to get the necessary parts of them into Blink as NavigationState earlier,
allowing them to follow the navigation.

 }
 
 void RenderFrameImpl::UpdateEncoding(WebFrame* frame,