PlzNavigate: add initial traces for new TBM benchmarks.

content/browser/loader/navigation_url_loader_impl.cc

Index: content/browser/loader/navigation_url_loader_impl.cc
diff --git a/content/browser/loader/navigation_url_loader_impl.cc b/content/browser/loader/navigation_url_loader_impl.cc
index 2cae9a6237240334d3d026ea6b1d5f5075744b9b..7e651094c1df045edfe8398f389bbf177814897b 100644
--- a/content/browser/loader/navigation_url_loader_impl.cc
+++ b/content/browser/loader/navigation_url_loader_impl.cc
@@ -6,6 +6,7 @@
 
 #include "base/bind.h"
 #include "base/location.h"
+#include "base/trace_event/trace_event.h"
 #include "content/browser/frame_host/navigation_request_info.h"
 #include "content/browser/loader/navigation_url_loader_delegate.h"
 #include "content/browser/loader/navigation_url_loader_impl_core.h"
@@ -25,6 +26,10 @@ NavigationURLLoaderImpl::NavigationURLLoaderImpl(
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
 
   core_ = new NavigationURLLoaderImplCore(weak_factory_.GetWeakPtr());
+  TRACE_EVENT_ASYNC_BEGIN_WITH_TIMESTAMP1(

[clamy, 2015/12/18 15:42:11]

This is called on the UI thread while the end happens on the IO thread. Is that
an issue?

[carlosk, 2015/12/18 17:23:21]

Nope, it's supported behavior:
https://code.google.com/p/chromium/codesearch#chromium/src/base/trace_event/c...

+      "navigation", "Navigation timeToResponseStarted", core_,
+      request_info->common_params.navigation_start.ToInternalValue(),
+      "FrameTreeNode id", request_info->frame_tree_node_id);
   ServiceWorkerNavigationHandleCore* service_worker_handle_core =
       service_worker_handle ? service_worker_handle->core() : nullptr;
   BrowserThread::PostTask(