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..9a3fc83a67bc2dd409c3521efc1c8610fba49311 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 ready to be committed in a renderer. This is
+  // the first point in time where a RenderFrameHost is associated with the
+  // navigation.

[nasko, 2015/09/16 21:37:50]

One thing I forgot to mention. Let's add a comment here saying that observers
that want to initialize any renderer side structures/state before the
RenderFrame is navigated, should use this method as opposed to
DidCommitNavigation, which is after the fact.

[clamy, 2015/09/16 22:19:39]

Done.

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