Moves RenderProcessHostImpl::init_time() to RenderProcessHost to fix broken test.

content/public/browser/render_process_host.h

Index: content/public/browser/render_process_host.h
diff --git a/content/public/browser/render_process_host.h b/content/public/browser/render_process_host.h
index c1b5c47e1d2fd5b38f90dab5febc685e76c0a7a9..4cd5390b021f9c38199a05fdb740b6add8f3abbc 100644
--- a/content/public/browser/render_process_host.h
+++ b/content/public/browser/render_process_host.h
@@ -229,6 +229,13 @@ class CONTENT_EXPORT RenderProcessHost : public IPC::Sender,
   // Returns the ServiceRegistry for this process.
   virtual ServiceRegistry* GetServiceRegistry() = 0;
 
+  // PlzNavigate
+  // Returns the time the first call to Init completed successfully (after a new
+  // renderer process was created); further calls to Init won't change this
+  // value.
+  // Note: Do not use! Will disappear after PlzNavitate is completed.
+  virtual const base::TimeTicks& GetInitTime() const = 0;

[nasko, 2014/10/20 19:53:57]

I like Avi's suggestion. Let's decorate the name with something like
ForNavigationTiming or anything you think is better.

[carlosk, 2014/10/21 12:23:54]

I suffixed it with ForPlzNavigate. IMO the odd sounding name should ring a bell
to anyone considering using it.

[nasko, 2014/10/21 14:13:55]

I agree with jam@ that using the project name is not really useful. Suffixing it
will already show some purpose, but we avoid using project names.

[carlosk, 2014/10/21 15:28:13]

OK, replaced it with GetInitTimeForNavigationMetrics (sounded better than
time-for-timing).

[nasko, 2014/10/21 15:56:14]

Ah, I like that better too!

+
   // Static management functions -----------------------------------------------
 
   // Flag to run the renderer in process.  This is primarily