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

Unified Diff: chrome/browser/autocomplete/history_quick_provider.cc

Issue 353223002: Omnibox: Fix URL-What-You-Typed Allowed-To-Be-Default-Match Issues (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove some test cases; add dcheck Created 6 years, 5 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
Index: chrome/browser/autocomplete/history_quick_provider.cc
diff --git a/chrome/browser/autocomplete/history_quick_provider.cc b/chrome/browser/autocomplete/history_quick_provider.cc
index 336a9c43850ddd2aa436720715f52a054f59f2a2..e63a9678db4d6f0d95b0c61f08d0d9ed84e419ea 100644
--- a/chrome/browser/autocomplete/history_quick_provider.cc
+++ b/chrome/browser/autocomplete/history_quick_provider.cc
@@ -9,6 +9,7 @@
#include "base/basictypes.h"
#include "base/command_line.h"
#include "base/i18n/break_iterator.h"
+#include "base/i18n/case_conversion.h"
Peter Kasting 2014/07/09 22:36:00 Nit: Is this needed?
Mark P 2014/07/09 23:12:23 Not anymore. Removed.
#include "base/logging.h"
#include "base/metrics/field_trial.h"
#include "base/metrics/histogram.h"
@@ -275,6 +276,9 @@ AutocompleteMatch HistoryQuickProvider::QuickMatchToACMatch(
match.allowed_to_be_default_match = match.inline_autocompletion.empty() ||
!PreventInlineAutocomplete(autocomplete_input_);
}
+ match.EnsureUWYTIsAllowedToBeDefault(
+ autocomplete_input_.canonicalized_url(),
+ TemplateURLServiceFactory::GetForProfile(profile_));
// Format the description autocomplete presentation.
match.description = info.title();

Powered by Google App Engine
This is Rietveld 408576698