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

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

Issue 211283005: Revert of Handle Search suggest subtypes for Shortcuts DB (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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/shortcuts_backend.cc
diff --git a/chrome/browser/autocomplete/shortcuts_backend.cc b/chrome/browser/autocomplete/shortcuts_backend.cc
index f8353b4d7487c317311dd5b52fc76a5c7541614a..b7d5d780a16a96bdf486113031e85ca63389d819 100644
--- a/chrome/browser/autocomplete/shortcuts_backend.cc
+++ b/chrome/browser/autocomplete/shortcuts_backend.cc
@@ -13,17 +13,14 @@
#include "base/guid.h"
#include "base/i18n/case_conversion.h"
#include "base/strings/string_util.h"
-#include "chrome/browser/autocomplete/autocomplete_input.h"
#include "chrome/browser/autocomplete/autocomplete_match.h"
#include "chrome/browser/autocomplete/autocomplete_result.h"
-#include "chrome/browser/autocomplete/base_search_provider.h"
#include "chrome/browser/chrome_notification_types.h"
#include "chrome/browser/history/history_notifications.h"
#include "chrome/browser/history/history_service.h"
#include "chrome/browser/history/shortcuts_database.h"
#include "chrome/browser/omnibox/omnibox_log.h"
#include "chrome/browser/profiles/profile.h"
-#include "chrome/common/autocomplete_match_type.h"
#include "chrome/common/chrome_constants.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/notification_details.h"
@@ -55,12 +52,16 @@
case AutocompleteMatchType::NAVSUGGEST:
return AutocompleteMatchType::HISTORY_URL;
- case AutocompleteMatchType::SEARCH_OTHER_ENGINE:
+ case AutocompleteMatchType::SEARCH_WHAT_YOU_TYPED:
+ case AutocompleteMatchType::SEARCH_SUGGEST:
+ case AutocompleteMatchType::SEARCH_SUGGEST_ENTITY:
+ case AutocompleteMatchType::SEARCH_SUGGEST_INFINITE:
+ case AutocompleteMatchType::SEARCH_SUGGEST_PERSONALIZED:
+ case AutocompleteMatchType::SEARCH_SUGGEST_PROFILE:
+ return AutocompleteMatchType::SEARCH_HISTORY;
+
+ default:
return type;
-
- default:
- return AutocompleteMatch::IsSearchType(type) ?
- AutocompleteMatchType::SEARCH_HISTORY : type;
}
}
@@ -70,8 +71,7 @@
// ShortcutsBackend -----------------------------------------------------------
ShortcutsBackend::ShortcutsBackend(Profile* profile, bool suppress_db)
- : profile_(profile),
- current_state_(NOT_INITIALIZED),
+ : current_state_(NOT_INITIALIZED),
no_db_access_(suppress_db) {
if (!suppress_db) {
db_ = new history::ShortcutsDatabase(
@@ -124,13 +124,13 @@
StartsWith(it->first, text_lowercase, true); ++it) {
if (match.destination_url == it->second.match_core.destination_url) {
UpdateShortcut(history::ShortcutsDatabase::Shortcut(
- it->second.id, text, MatchToMatchCore(match, profile_), now,
+ it->second.id, text, MatchToMatchCore(match), now,
it->second.number_of_hits + 1));
return;
}
}
AddShortcut(history::ShortcutsDatabase::Shortcut(
- base::GenerateGUID(), text, MatchToMatchCore(match, profile_), now, 1));
+ base::GenerateGUID(), text, MatchToMatchCore(match), now, 1));
}
ShortcutsBackend::~ShortcutsBackend() {
@@ -138,24 +138,12 @@
// static
history::ShortcutsDatabase::Shortcut::MatchCore
- ShortcutsBackend::MatchToMatchCore(const AutocompleteMatch& match,
- Profile* profile) {
- const base::string16& suggestion = match.search_terms_args->search_terms;
- const AutocompleteMatch::Type match_type = GetTypeForShortcut(match.type);
- const AutocompleteMatch& normalized_match =
- AutocompleteMatch::IsSpecializedSearchType(match.type) ?
- BaseSearchProvider::CreateSearchSuggestion(
- suggestion, match_type,
- (match.transition == content::PAGE_TRANSITION_KEYWORD),
- match.GetTemplateURL(profile, false)) :
- match;
+ ShortcutsBackend::MatchToMatchCore(const AutocompleteMatch& match) {
return history::ShortcutsDatabase::Shortcut::MatchCore(
- normalized_match.fill_into_edit, normalized_match.destination_url,
- normalized_match.contents,
- StripMatchMarkers(normalized_match.contents_class),
- normalized_match.description,
- StripMatchMarkers(normalized_match.description_class),
- normalized_match.transition, match_type, normalized_match.keyword);
+ match.fill_into_edit, match.destination_url, match.contents,
+ StripMatchMarkers(match.contents_class), match.description,
+ StripMatchMarkers(match.description_class), match.transition,
+ GetTypeForShortcut(match.type), match.keyword);
}
void ShortcutsBackend::ShutdownOnUIThread() {
« no previous file with comments | « chrome/browser/autocomplete/shortcuts_backend.h ('k') | chrome/browser/autocomplete/shortcuts_backend_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698