Don't report origin changes during loading.

chrome/browser/search_engines/template_url_service.h

 // 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.
 
 #ifndef CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_SERVICE_H_
 #define CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_SERVICE_H_
 
 #include <list>
 #include <map>
 #include <set>
@@ skipping 444 matching lines @@
     // All changes that don't fall into another category; we can't reorder the
     // list for clarity as this would screw up stat collection.
     DSP_CHANGE_OTHER,
     // Changed through "Profile Reset" feature.
     DSP_CHANGE_PROFILE_RESET,
     // Changed by an extension through the Override Settings API.
     DSP_CHANGE_OVERRIDE_SETTINGS_EXTENSION,
     // New DSP during database/prepopulate data load, which was not previously
     // in the known engine set, and with no previous value in prefs.  The
     // typical time to see this is during first run.
     DSP_CHANGE_NEW_ENGINE_NO_PREFS,

[Peter Kasting, 2014/05/21 19:38:41]

This value no longer seems to be in use.  Maybe it should be removed.

[erikwright (departed), 2014/05/21 20:38:48]

Several of the values are unused now. I can remove them (in fact, I have
prepared a CL to mark them as deprecated in histograms.xml) but I would need to
insert placeholders to keep UMA data consistent.

[Peter Kasting, 2014/05/21 20:45:13]

That still seems preferable.  OK to do it separately if you want.

+    // The DSP value is being applied while loading preferences from disk. This
+    // is not actually a change (we don't report this).
+    DSP_CHANGE_INITIAL_LOAD,
     // Boundary value.
     DSP_CHANGE_MAX,
   };
 
   // Helper functor for FindMatchingKeywords(), for finding the range of
   // keywords which begin with a prefix.
   class LessWithPrefix;
 
   void Init(const Initializer* initializers, int num_initializers);
 
@@ skipping 303 matching lines @@
 
   scoped_ptr<GoogleURLTracker::Subscription> google_url_updated_subscription_;
 
   // Used to disable the prepopulated search engines in tests.
   static bool g_fallback_search_engines_disabled;
 
   DISALLOW_COPY_AND_ASSIGN(TemplateURLService);
 };
 
 #endif  // CHROME_BROWSER_SEARCH_ENGINES_TEMPLATE_URL_SERVICE_H_