Omnibox: Log Elapsed Time Since Last Time The Default Match Changed

chrome/browser/metrics/metrics_log.cc

Index: chrome/browser/metrics/metrics_log.cc
diff --git a/chrome/browser/metrics/metrics_log.cc b/chrome/browser/metrics/metrics_log.cc
index 9ff3957707f38494c7d84f12f5812911a968fcd3..9110717fbdabf99bfb839a86193527c86c1d0116 100644
--- a/chrome/browser/metrics/metrics_log.cc
+++ b/chrome/browser/metrics/metrics_log.cc
@@ -978,6 +978,8 @@ void MetricsLog::RecordOmniboxOpenedURL(const AutocompleteLog& log) {
     omnibox_event->set_typing_duration_ms(
         log.elapsed_time_since_user_first_modified_omnibox.InMilliseconds());
   }
+  omnibox_event->set_duration_since_last_default_match_update_ms(
+      log.elapsed_time_since_user_first_modified_omnibox.InMilliseconds());

[Ilya Sherman, 2012/11/20 01:44:30]

Looks like you're using the same value here and on line 979 -- I'm assuming you
meant to use a different one here?

[Mark P, 2012/11/20 17:34:41]

Good catch!  Fixed.

   omnibox_event->set_current_page_classification(
       log.current_page_classification);
   omnibox_event->set_input_type(AsOmniboxEventInputType(log.input_type));