Rename WebContentsObserver::NavigateToPendingEntry to DidStartNavigationToPendingEntry.

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 fbc5206f1f90af2ef4534942ae83bfe8d38a3774..dbca456ee01e715cead556d864d5b2c9f6c33dc3 100644
--- a/content/public/browser/web_contents_observer.h
+++ b/content/public/browser/web_contents_observer.h
@@ -80,7 +80,7 @@ class CONTENT_EXPORT WebContentsObserver : public IPC::Listener,
       RenderViewHost* render_view_host) {}
 
   // This method is invoked right after the navigation was initiated.
-  virtual void NavigateToPendingEntry(
+  virtual void DidStartNavigationToPendingEntry(

[Charlie Reis, 2013/11/22 23:29:10]

Much better, thanks.  Let's try to capture some of the reasoning for the name in
the comment, with something like:

This method is invoked after the browser process starts a navigation to a
pending NavigationEntry.  It is not called for renderer-initiated navigations
unless they are sent to the browser process via OpenURL.  It may be called
multiple times for a given navigation, such as a typed URL followed by a
cross-process client or server redirect.

[nasko, 2013/11/23 00:17:25]

Done.

       const GURL& url,
       NavigationController::ReloadType reload_type) {}