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

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: Address comments 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 0931a0cc72da93d253c6fc0ec7a93061b93bf5fc..b56e1b5c8ffa114efd16ec203fbfa82c79367c39 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);
+ }
}
}
}
« no previous file with comments | « components/search_engines/template_url_service.h ('k') | components/search_engines/template_url_service_sync_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698