Changes PlzNavitate histograms to try and simplify their multi-modal characteristic.

content/browser/renderer_host/render_process_host_impl.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_BROWSER_RENDERER_HOST_BROWSER_RENDER_PROCESS_HOST_IMPL_H_
 #define CONTENT_BROWSER_RENDERER_HOST_BROWSER_RENDER_PROCESS_HOST_IMPL_H_
 
 #include <map>
 #include <queue>
 #include <string>
@@ skipping 121 matching lines @@
       base::Callback<void(const std::string&)> callback) override;
   virtual WebRtcStopRtpDumpCallback StartRtpDump(
       bool incoming,
       bool outgoing,
       const WebRtcRtpPacketCallback& packet_callback) override;
 #endif
   virtual void ResumeDeferredNavigation(const GlobalRequestID& request_id)
       override;
   virtual void NotifyTimezoneChange() override;
   virtual ServiceRegistry* GetServiceRegistry() override;
+  virtual const base::TimeTicks& GetInitTime() const;

[nasko, 2014/10/13 15:55:24]

This method is no longer inherited, so why is it virtual?
Do we plan to keep this method just for some data collection of for the long
run? 

[carlosk, 2014/10/13 18:01:46]

Done, not virtual anymore. I forgot to update it after that change. I'm also
flagging it as being a PlzNavigate thingie.

 
   // IPC::Sender via RenderProcessHost.
   virtual bool Send(IPC::Message* msg) override;
 
   // IPC::Listener via RenderProcessHost.
   virtual bool OnMessageReceived(const IPC::Message& msg) override;
   virtual void OnChannelConnected(int32 peer_pid) override;
   virtual void OnChannelError() override;
   virtual void OnBadMessageReceived(const IPC::Message& message) override;
 
@@ skipping 212 matching lines @@
   // The filter for MessagePort messages coming from the renderer.
   scoped_refptr<MessagePortMessageFilter> message_port_message_filter_;
 
   // Used in single-process mode.
   scoped_ptr<base::Thread> in_process_renderer_;
 
   // True after Init() has been called. We can't just check channel_ because we
   // also reset that in the case of process termination.
   bool is_initialized_;
 
+  // The time at which the first call to Init happened and the child renderer
+  // process was created.
+  base::TimeTicks init_time_;
+
   // Used to launch and terminate the process without blocking the UI thread.
   scoped_ptr<ChildProcessLauncher> child_process_launcher_;
 
   // Messages we queue while waiting for the process handle.  We queue them here
   // instead of in the channel so that we ensure they're sent after init related
   // messages that are sent once the process handle is available.  This is
   // because the queued messages may have dependencies on the init messages.
   std::queue<IPC::Message*> queued_messages_;
 
   // The globally-unique identifier for this RPH.
@@ skipping 72 matching lines @@
   base::TimeTicks survive_for_worker_start_time_;
 
   base::WeakPtrFactory<RenderProcessHostImpl> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(RenderProcessHostImpl);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_RENDERER_HOST_BROWSER_RENDER_PROCESS_HOST_IMPL_H_