Don't persist and sync omnibox extension keywords.

components/search_engines/search_host_to_urls_map.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/search_engines/search_host_to_urls_map.h"
 
 #include "base/memory/scoped_ptr.h"
 #include "components/search_engines/template_url.h"
 
 SearchHostToURLsMap::SearchHostToURLsMap()
     : initialized_(false) {
 }
 
 SearchHostToURLsMap::~SearchHostToURLsMap() {
 }
 
 void SearchHostToURLsMap::Init(
     const TemplateURLService::TemplateURLVector& template_urls,
     const SearchTermsData& search_terms_data) {
   DCHECK(!initialized_);
   initialized_ = true;  // Set here so Add doesn't assert.
   Add(template_urls, search_terms_data);
 }
 
 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())
     return;
 
   host_to_urls_map_[url.host()].insert(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) {
     TemplateURLSet::iterator url_set_iterator = i->second.find(template_url);
     if (url_set_iterator != i->second.end()) {
       i->second.erase(url_set_iterator);
       if (i->second.empty())
         host_to_urls_map_.erase(i);
       // A given TemplateURL only occurs once in the map. As soon as we find the
       // entry, stop.
@@ skipping 22 matching lines @@
   return &urls_for_host->second;
 }
 
 void SearchHostToURLsMap::Add(
     const TemplateURLService::TemplateURLVector& template_urls,
     const SearchTermsData& search_terms_data) {
   for (TemplateURLService::TemplateURLVector::const_iterator i(
        template_urls.begin()); i != template_urls.end(); ++i)
     Add(*i, search_terms_data);
 }