Fix a race condition in preference metric reporting.

chrome/browser/prefs/chrome_pref_service_factory.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/prefs/chrome_pref_service_factory.h"
 
 #include "base/bind.h"
 #include "base/debug/trace_event.h"
 #include "base/file_util.h"
 #include "base/files/file_path.h"
 #include "base/metrics/histogram.h"
 #include "base/prefs/default_pref_store.h"
 #include "base/prefs/json_pref_store.h"
 #include "base/prefs/pref_notifier_impl.h"
 #include "base/prefs/pref_registry.h"
 #include "base/prefs/pref_service.h"
 #include "base/prefs/pref_value_store.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/prefs/command_line_pref_store.h"
+#include "chrome/browser/prefs/pref_hash_filter.h"
+#include "chrome/browser/prefs/pref_hash_store.h"
 #include "chrome/browser/prefs/pref_model_associator.h"
 #include "chrome/browser/prefs/pref_service_syncable.h"
 #include "chrome/browser/prefs/pref_service_syncable_factory.h"
 #include "chrome/browser/ui/profile_error_dialog.h"
 #include "components/user_prefs/pref_registry_syncable.h"
 #include "content/public/browser/browser_context.h"
 #include "content/public/browser/browser_thread.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 
@@ skipping 43 matching lines @@
 #endif
   }
 }
 
 void PrepareBuilder(
     PrefServiceSyncableFactory* factory,
     const base::FilePath& pref_filename,
     base::SequencedTaskRunner* pref_io_task_runner,
     policy::PolicyService* policy_service,
     ManagedUserSettingsService* managed_user_settings,
+    scoped_ptr<PrefHashStore> pref_hash_store,
     const scoped_refptr<PrefStore>& extension_prefs,
     bool async) {
 #if defined(OS_LINUX)
   // We'd like to see what fraction of our users have the preferences
   // stored on a network file system, as we've had no end of troubles
   // with NFS/AFS.
   // TODO(evanm): remove this once we've collected state.
   file_util::FileSystemType fstype;
   if (file_util::GetFileSystemType(pref_filename.DirName(), &fstype)) {
     UMA_HISTOGRAM_ENUMERATION("PrefService.FileSystemType",
@@ skipping 22 matching lines @@
         make_scoped_refptr(new SupervisedUserPrefStore(managed_user_settings)));
   }
 #endif
 
   factory->set_async(async);
   factory->set_extension_prefs(extension_prefs);
   factory->set_command_line_prefs(
       make_scoped_refptr(
           new CommandLinePrefStore(CommandLine::ForCurrentProcess())));
   factory->set_read_error_callback(base::Bind(&HandleReadError));
+
+  scoped_ptr<PrefFilter> pref_filter;
+  if (pref_hash_store)
+    pref_filter.reset(new PrefHashFilter(pref_hash_store.Pass()));

[gab, 2013/12/17 01:44:09]

@erikwright: Fixed this for you as discussed, will CQ now.

I think this highlights the fact that it could be better to ask the API's client
to pass a PrefHashFilter in here directly; perhaps even moving the params of
PrefHashStore into PrefHashFilter's constructor, making PrefHashStore an
implementation detail of PrefHashFilter (akin to PrefHashCalculator vs
PrefHashStore).

Let's consider this change for a follow-up CL.

   factory->set_user_prefs(
-      new JsonPrefStore(pref_filename, pref_io_task_runner));
+      new JsonPrefStore(
+          pref_filename,
+          pref_io_task_runner,
+          pref_filter.Pass()));
 }
 
 }  // namespace
 
 namespace chrome_prefs {
 
 scoped_ptr<PrefService> CreateLocalState(
     const base::FilePath& pref_filename,
     base::SequencedTaskRunner* pref_io_task_runner,
     policy::PolicyService* policy_service,
     const scoped_refptr<PrefRegistry>& pref_registry,
     bool async) {
   PrefServiceSyncableFactory factory;
   PrepareBuilder(&factory,
                  pref_filename,
                  pref_io_task_runner,
                  policy_service,
                  NULL,
+                 scoped_ptr<PrefHashStore>(),
                  NULL,
                  async);
   return factory.Create(pref_registry.get());
 }
 
 scoped_ptr<PrefServiceSyncable> CreateProfilePrefs(
     const base::FilePath& pref_filename,
     base::SequencedTaskRunner* pref_io_task_runner,
     policy::PolicyService* policy_service,
     ManagedUserSettingsService* managed_user_settings,
+    scoped_ptr<PrefHashStore> pref_hash_store,
     const scoped_refptr<PrefStore>& extension_prefs,
     const scoped_refptr<user_prefs::PrefRegistrySyncable>& pref_registry,
     bool async) {
   TRACE_EVENT0("browser", "chrome_prefs::CreateProfilePrefs");
   PrefServiceSyncableFactory factory;
   PrepareBuilder(&factory,
                  pref_filename,
                  pref_io_task_runner,
                  policy_service,
                  managed_user_settings,
+                 pref_hash_store.Pass(),
                  extension_prefs,
                  async);
   return factory.CreateSyncable(pref_registry.get());
 }
 
 }  // namespace chrome_prefs