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

Unified Diff: components/search_engines/template_url_service.cc

Issue 513133002: Move TemplateURLService related tests to components/search_engines (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 4 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: components/search_engines/template_url_service.cc
diff --git a/components/search_engines/template_url_service.cc b/components/search_engines/template_url_service.cc
index 1fcc7041380cc68f3acbc6db73afe56f57ce9f4e..ea891d869fc43136639445b6b2d82f985dc1fede 100644
--- a/components/search_engines/template_url_service.cc
+++ b/components/search_engines/template_url_service.cc
@@ -1408,14 +1408,6 @@ SyncDataMap TemplateURLService::CreateGUIDToSyncDataMap(
return data_map;
}
-void TemplateURLService::SetKeywordSearchTermsForURL(
- const TemplateURL* t_url,
- const GURL& url,
- const base::string16& term) {
- if (client_)
- client_->SetKeywordSearchTermsForURL(url, t_url->id(), term);
-}
-
void TemplateURLService::Init(const Initializer* initializers,
int num_initializers) {
if (client_)
@@ -1747,7 +1739,10 @@ void TemplateURLService::UpdateKeywordSearchTermsForURL(
// count is boosted.
AddTabToSearchVisit(**i);
}
- SetKeywordSearchTermsForURL(*i, details.url, search_terms);
+ if (client_) {
+ client_->SetKeywordSearchTermsForURL(
+ details.url, (*i)->id(), search_terms);
+ }
}
}
}

Powered by Google App Engine
This is Rietveld 408576698