Use the DefaultSearchManager as the exclusive authority on DSE, ignoring Web Data.

chrome/browser/search_engines/template_url.cc

 // Copyright (c) 2012 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 "chrome/browser/search_engines/template_url.h"
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 1258 matching lines @@
     if (TryEncoding(search_terms_args.search_terms,
                     search_terms_args.original_query, i->c_str(),
                     is_in_query, encoded_terms, encoded_original_query)) {
       *input_encoding = *i;
       return;
     }
   }
   NOTREACHED();
 }
 
-void TemplateURL::CopyFrom(const TemplateURL& other) {
-  if (this == &other)
+void TemplateURL::CopyFrom(const TemplateURLData& other) {
+  if (&data_ == &other)
     return;
 
-  profile_ = other.profile_;

[Peter Kasting, 2014/05/03 00:28:18]

Not copying this seems highly error-prone.  The profile dictates the meaning of
a number of other fields.

The old prototype made much more sense from a "how TemplateURLs work"
perspective.

-  data_ = other.data_;
+  data_ = other;
   url_ref_.InvalidateCachedValues();
   suggestions_url_ref_.InvalidateCachedValues();
   instant_url_ref_.InvalidateCachedValues();
-  SetPrepopulateId(other.data_.prepopulate_id);
+  SetPrepopulateId(other.prepopulate_id);
 }
 
 void TemplateURL::SetURL(const std::string& url) {
   data_.SetURL(url);
   url_ref_.InvalidateCachedValues();
 }
 
 void TemplateURL::SetPrepopulateId(int id) {
   data_.prepopulate_id = id;
   const bool prepopulated = id > 0;
@@ skipping 31 matching lines @@
       // patterns. This means that given patterns
       //    [ "http://foo/#q={searchTerms}", "http://foo/?q={searchTerms}" ],
       // calling ExtractSearchTermsFromURL() on "http://foo/?q=bar#q=' would
       // return false. This is important for at least Google, where such URLs
       // are invalid.
       return !search_terms->empty();
     }
   }
   return false;
 }