Remove WebContentsObserver::DidCommitNavigation

content/browser/frame_host/navigation_request.cc

Index: content/browser/frame_host/navigation_request.cc
diff --git a/content/browser/frame_host/navigation_request.cc b/content/browser/frame_host/navigation_request.cc
index a99074f2ce3aeb56440b44cc0e9530f0f0c1db39..b38889847121b6d9bf86418d4ab80efae1d71021 100644
--- a/content/browser/frame_host/navigation_request.cc
+++ b/content/browser/frame_host/navigation_request.cc
@@ -204,6 +204,8 @@ void NavigationRequest::CreateNavigationHandle(NavigatorDelegate* delegate) {
 void NavigationRequest::TransferNavigationHandleOwnership(
     RenderFrameHostImpl* render_frame_host) {
   render_frame_host->SetNavigationHandle(navigation_handle_.Pass());
+  render_frame_host->navigation_handle()->ReadyToCommitNavigation(

[nasko, 2015/09/18 16:42:16]

Why here instead of right before CommitNavigation call in OnResponseStarted? It
isn't immediately clear why this is correct.

[clamy, 2015/09/18 20:37:33]

Because we don't have a renderer prior to the call to CommitNavigation. And
after the call this request is destroyed. The handle is transfered before any
IPC is sent.

+      render_frame_host);
 }
 
 void NavigationRequest::OnRequestRedirected(