Remove frame ID from DidCommitProvisionalLoad.

content/public/browser/render_frame_host.h

Index: content/public/browser/render_frame_host.h
diff --git a/content/public/browser/render_frame_host.h b/content/public/browser/render_frame_host.h
index 0a56349141aa023cdb2e775c12362875f0d3b7e6..b661a1ab43635ea96e06327ce0e806bb31e61daf 100644
--- a/content/public/browser/render_frame_host.h
+++ b/content/public/browser/render_frame_host.h
@@ -27,14 +27,17 @@ class CONTENT_EXPORT RenderFrameHost : public IPC::Listener,
 
   virtual ~RenderFrameHost() {}
 
+  // Returns the route id for this frame.
+  virtual int GetRoutingID() = 0;

[jam, 2014/03/03 16:08:04]

nit: if you're changing this order, please also update the impl files

[Charlie Reis, 2014/03/03 17:05:23]

I already did, right?  Or did I put it in the wrong place?

+
+  // Returns the SiteInstance grouping all RenderFrameHosts that have script
+  // access to this RenderFrameHost, and must therefore live in the same
+  // process.
   virtual SiteInstance* GetSiteInstance() = 0;
 
   // Returns the process for this frame.
   virtual RenderProcessHost* GetProcess() = 0;
 
-  // Returns the route id for this frame.
-  virtual int GetRoutingID() = 0;
-
   // Returns the current RenderFrameHost of the parent frame, or NULL if there
   // is no parent. The result may be in a different process than the current
   // RenderFrameHost.