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

Unified Diff: chrome/browser/autocomplete/search_provider.h

Issue 18878007: Omnibox: Make the Controller Reorder Matches for Inlining (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remove another SearchProvider constraint not needed in reorder mode Created 7 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/search_provider.h
diff --git a/chrome/browser/autocomplete/search_provider.h b/chrome/browser/autocomplete/search_provider.h
index f01fdb67aaca1725a2f1fa0b2e5460f0c3166f4a..e48162d7fb4fa716c0197bbe9db9fda561dc4f49 100644
--- a/chrome/browser/autocomplete/search_provider.h
+++ b/chrome/browser/autocomplete/search_provider.h
@@ -379,7 +379,8 @@ class SearchProvider : public AutocompleteProvider,
bool IsTopMatchNavigationInKeywordMode() const;
bool IsTopMatchScoreTooLow() const;
bool IsTopMatchHighRankSearchForURL() const;
- bool IsTopMatchNotInlinable() const;
+ bool IsTopMatchNotAllowedToBeDefaultMatch() const;
+ bool LacksDefaultMatch() const;
// Updates |matches_| from the latest results; applies calculated relevances
// if suggested relevances cause undesriable behavior. Updates |done_|.
@@ -534,6 +535,10 @@ class SearchProvider : public AutocompleteProvider,
// If true, no search history query suggestions will be offered.
bool disable_search_history_;
+ // True if the omnibox will reorder matches as necessary to make the top
+ // one something that is allowed to be the default match.
+ bool omnibox_will_reorder_for_legal_default_match_;
+
DISALLOW_COPY_AND_ASSIGN(SearchProvider);
};

Powered by Google App Engine
This is Rietveld 408576698