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

chrome/browser/search_engines/template_url.cc

Index: chrome/browser/search_engines/template_url.cc
diff --git a/chrome/browser/search_engines/template_url.cc b/chrome/browser/search_engines/template_url.cc
index 501b7989e44816bed5d46828a94a22b1aefe07e6..a65b43b9458c070f9715042fbad861b2d3d404b8 100644
--- a/chrome/browser/search_engines/template_url.cc
+++ b/chrome/browser/search_engines/template_url.cc
@@ -1276,16 +1276,15 @@ void TemplateURL::EncodeSearchTerms(
   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) {