Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(211)

Unified Diff: chrome/browser/ui/omnibox/omnibox_edit_model.cc

Issue 314113009: Omnibox Metrics: Make |completed_length| field sane (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chrome/browser/omnibox/omnibox_log.h ('k') | components/metrics/proto/omnibox_event.proto » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/omnibox/omnibox_edit_model.cc
diff --git a/chrome/browser/ui/omnibox/omnibox_edit_model.cc b/chrome/browser/ui/omnibox/omnibox_edit_model.cc
index cbca6240bba7a630bb689869f37effd2fb43b1ca..1f6de5b89b38a3358cc9f408c0927d8564cd9cb5 100644
--- a/chrome/browser/ui/omnibox/omnibox_edit_model.cc
+++ b/chrome/browser/ui/omnibox/omnibox_edit_model.cc
@@ -751,7 +751,8 @@ void OmniboxEditModel::OpenMatch(AutocompleteMatch match,
-1, // don't yet know tab ID; set later if appropriate
ClassifyPage(),
elapsed_time_since_user_first_modified_omnibox,
- match.inline_autocompletion.length(),
+ match.allowed_to_be_default_match ? match.inline_autocompletion.length() :
+ base::string16::npos,
elapsed_time_since_last_change_to_default_match,
(!popup_model()->IsOpen() || !pasted_text.empty()) ?
fake_single_entry_result : result());
« no previous file with comments | « chrome/browser/omnibox/omnibox_log.h ('k') | components/metrics/proto/omnibox_event.proto » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698