Add time from navigation_start to NTP LogEvent

chrome/renderer/searchbox/searchbox.cc

 // Copyright 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.
 
 #include "chrome/renderer/searchbox/searchbox.h"
 
 #include <string>
 
 #include "base/strings/string_number_conversions.h"
 #include "base/strings/string_util.h"
 #include "base/strings/utf_string_conversions.h"
+#include "base/time/time.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/favicon/favicon_url_parser.h"
 #include "chrome/common/omnibox_focus_state.h"
 #include "chrome/common/render_messages.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/renderer/searchbox/searchbox_extension.h"
 #include "components/favicon_base/favicon_types.h"
+#include "content/public/renderer/render_frame.h"
 #include "content/public/renderer/render_view.h"
 #include "net/base/escape.h"
 #include "third_party/WebKit/public/web/WebDocument.h"
 #include "third_party/WebKit/public/web/WebFrame.h"
+#include "third_party/WebKit/public/web/WebLocalFrame.h"
+#include "third_party/WebKit/public/web/WebPerformance.h"
 #include "third_party/WebKit/public/web/WebView.h"
 #include "url/gurl.h"
 
 namespace {
 
 // The size of the InstantMostVisitedItem cache.
 const size_t kMaxInstantMostVisitedItemCacheSize = 100;
 
 // Returns true if items stored in |old_item_id_pairs| and |new_items| are
 // equal.
@@ skipping 116 matching lines @@
     display_instant_results_(false),
     most_visited_items_cache_(kMaxInstantMostVisitedItemCacheSize),
     query_(),
     start_margin_(0) {
 }
 
 SearchBox::~SearchBox() {
 }
 
 void SearchBox::LogEvent(NTPLoggingEventType event) {
+  // navigation_start in ms.
+  uint64 start = 1000 *
+                 render_view()
+                     ->GetMainRenderFrame()
+                     ->GetWebFrame()
+                     ->performance()
+                     .navigationStart();
+  uint64 now = (base::Time::Now() - base::Time::UnixEpoch()).InMilliseconds();

[kmadhusu, 2015/03/16 21:58:48]

I am not an expert to decide which function to use here. Based on the header
comments specified in
https://code.google.com/p/chromium/codesearch#chromium/src/base/time/time.h&t...,
"TimeTicks"  is guaranteed not to decrease whereas "Time" may actually decrease
or jump. I just wanted to make sure that you thought about it before using it
here. 

[fserb, 2015/03/16 22:05:32]

yeah, I saw that. But since I'm always using (now - epoch), it's pretty safe
either way. The reason I used TimeTicks (and maybe I should keep it) is because
navigationStart uses it too.

[kmadhusu, 2015/03/17 00:10:40]

Sure.

[fserb, 2015/03/17 15:54:49]

Acknowledged.

+  DCHECK(now >= start);
+  uint64 delta = now - start;
+  DVLOG(1) << now << " - " << start << " = " << delta;
   render_view()->Send(new ChromeViewHostMsg_LogEvent(
-      render_view()->GetRoutingID(), page_seq_no_, event));
+      render_view()->GetRoutingID(), page_seq_no_, event, delta));
 }
 
 void SearchBox::LogMostVisitedImpression(int position,
                                          const base::string16& provider) {
   render_view()->Send(new ChromeViewHostMsg_LogMostVisitedImpression(
       render_view()->GetRoutingID(), page_seq_no_, position, provider));
 }
 
 void SearchBox::LogMostVisitedNavigation(int position,
                                          const base::string16& provider) {
@@ skipping 312 matching lines @@
 
 void SearchBox::Reset() {
   query_.clear();
   embedded_search_request_params_ = EmbeddedSearchRequestParams();
   suggestion_ = InstantSuggestion();
   start_margin_ = 0;
   is_focused_ = false;
   is_key_capture_enabled_ = false;
   theme_info_ = ThemeBackgroundInfo();
 }