New TimeToFirstURLJob* navigation metrics now work with PlzNavigate.

content/browser/frame_host/navigator.h

 // Copyright 2013 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_FRAME_HOST_NAVIGATOR_H_
 #define CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/time/time.h"
 #include "content/common/content_export.h"
@@ skipping 122 matching lines @@
                                 ResourceResponse* response,
                                 scoped_ptr<StreamHandle> body);
 
   // PlzNavigate
   // Cancel a NavigationRequest for |frame_tree_node|. Called when
   // |frame_tree_node| is destroyed.
   virtual void CancelNavigation(FrameTreeNode* frame_tree_node) {}
 
   // Called when the first resource request for a given navigation is executed
   // so that it can be tracked into an histogram.
+  // PlzNavigate: Called when a navigation is about to make its first potential
+  // network request so that the timestamp can be recorded into an histogram.
+  // TODO(carlosk): once please navigate is the only navigation implementation

[davidben, 2015/01/28 21:33:26]

Nit: please navigate -> PlzNavigate is probably clearer it's referring to the
same thing.

[carlosk, 2015/01/30 14:01:47]

Done. LoL!

+  // remove the URL parameter and rename this method to better suit its naming
+  // conventions.
   virtual void LogResourceRequestTime(
     base::TimeTicks timestamp, const GURL& url) {};
 
+  virtual void LogAboutToBeginNavigation(base::TimeTicks timestamp) {};
+
   // Called to record the time it took to execute the before unload hook for the
   // current navigation.
   virtual void LogBeforeUnloadTime(
       const base::TimeTicks& renderer_before_unload_start_time,
       const base::TimeTicks& renderer_before_unload_end_time) {}
 
   // PlzNavigate
   // Returns whether there is an ongoing navigation waiting for the BeforeUnload
   // event to execute in the renderer process.
   virtual bool IsWaitingForBeforeUnloadACK(FrameTreeNode* frame_tree_node);
 
  protected:
   friend class base::RefCounted<Navigator>;
   virtual ~Navigator() {}
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_FRAME_HOST_NAVIGATOR_H_