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

Unified Diff: components/search_engines/search_host_to_urls_map.cc

Issue 684493002: Don't persist and sync omnibox extension keywords. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: change style to c++11 Created 6 years, 2 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/search_host_to_urls_map.cc
diff --git a/components/search_engines/search_host_to_urls_map.cc b/components/search_engines/search_host_to_urls_map.cc
index e309b0749e363255093c649609bce62fa7100e46..559b28784a9019767317de2ea28f4e31b9f39031 100644
--- a/components/search_engines/search_host_to_urls_map.cc
+++ b/components/search_engines/search_host_to_urls_map.cc
@@ -26,6 +26,7 @@ void SearchHostToURLsMap::Add(TemplateURL* template_url,
const SearchTermsData& search_terms_data) {
DCHECK(initialized_);
DCHECK(template_url);
+ DCHECK(template_url->GetType() != TemplateURL::OMNIBOX_API_EXTENSION);
Peter Kasting 2014/10/29 18:55:51 Nit: DCHECK_NE (2 places)
vasilii 2014/11/03 15:14:30 Done.
const GURL url(template_url->GenerateSearchURL(search_terms_data));
if (!url.is_valid() || !url.has_host())
@@ -37,6 +38,7 @@ void SearchHostToURLsMap::Add(TemplateURL* template_url,
void SearchHostToURLsMap::Remove(TemplateURL* template_url) {
DCHECK(initialized_);
DCHECK(template_url);
+ DCHECK(template_url->GetType() != TemplateURL::OMNIBOX_API_EXTENSION);
for (HostToURLsMap::iterator i = host_to_urls_map_.begin();
i != host_to_urls_map_.end(); ++i) {

Powered by Google App Engine
This is Rietveld 408576698