PlzNavigate: fix timing issue in app window creation

content/public/browser/web_contents_observer.h

Index: content/public/browser/web_contents_observer.h
diff --git a/content/public/browser/web_contents_observer.h b/content/public/browser/web_contents_observer.h
index d310cb229e485af2d66577fff629b5c71e791334..610662a420d87289aff92eba5e8af9fa3d03db33 100644
--- a/content/public/browser/web_contents_observer.h
+++ b/content/public/browser/web_contents_observer.h
@@ -121,8 +121,9 @@ class CONTENT_EXPORT WebContentsObserver : public IPC::Listener,
 
   // Called when a navigation started in the WebContents. |navigation_handle|
   // is unique to a specific navigation. The same |navigation_handle| will be
-  // provided on subsequent calls to DidRedirect/Commit/FinishNavigation
-  // related to this navigation.
+  // provided on subsequent calls to
+  // DidRedirect/Commit/FinishNavigation/AboutToCommitNavigation related to
+  // this navigation.
   //
   // Note that this is fired by navigations in any frame of the WebContents,
   // not just the main frame.
@@ -139,6 +140,12 @@ class CONTENT_EXPORT WebContentsObserver : public IPC::Listener,
   // Called when a navigation encountered a server redirect.
   virtual void DidRedirectNavigation(NavigationHandle* navigation_handle) {}
 
+  // PlzNavigate
+  // Called when the navigation is about to be committed in a renderer. This is
+  // the first point in time where a RenderFrameHost is associated with the
+  // navigation.
+  virtual void AboutToCommitNavigation(NavigationHandle* navigation_handle) {}

[nasko, 2015/09/16 20:19:36]

I think ReadyToCommitNavigation is a bit more precise. I find the "about to do
something" a bit more ambiguous.

[clamy, 2015/09/16 21:32:34]

Agreed. Changed the name.

+
   // Called when a navigation was committed.
   virtual void DidCommitNavigation(NavigationHandle* navigation_handle) {}