Simple code cleanup.

Passed PrefRegistry and PrefRegistrySyncable by raw pointer instead of reference
to scoped_ptr. PrefRegistry and PrefRegistrySyncable have an internal reference
count. There is no need to use a double indirection when passing them around.

Used base::MakeRefCounted instead of naked 'new' operator.

Change-Id: I53a0fdfc99b7ef3dd32b191664520659c2585fa5
Reviewed-on: https://chromium-review.googlesource.com/892582Reviewed-by: Scott Violet <sky@chromium.org>
Commit-Queue: François Degros <fdegros@chromium.org>
Cr-Commit-Position: refs/heads/master@{#533087}

chrome/browser/browser_process_impl.cc

@@ -1035,7 +1035,7 @@ void BrowserProcessImpl::CreateLocalState() {
 
   base::FilePath local_state_path;
   CHECK(PathService::Get(chrome::FILE_LOCAL_STATE, &local_state_path));
-  auto pref_registry = base::MakeRefCounted<PrefRegistrySimple>();
+  const auto pref_registry = base::MakeRefCounted<PrefRegistrySimple>();
 
   // Register local state preferences.
   RegisterLocalState(pref_registry.get());
@@ -1044,7 +1044,7 @@ void BrowserProcessImpl::CreateLocalState() {
   delegate->InitPrefRegistry(pref_registry.get());
   local_state_ = chrome_prefs::CreateLocalState(
       local_state_path, local_state_task_runner_.get(), policy_service(),
-      pref_registry, false, std::move(delegate));
+      pref_registry.get(), false, std::move(delegate));
   DCHECK(local_state_);
 
   pref_change_registrar_.Init(local_state_.get());

chrome/browser/chrome_browser_main.cc

@@ -390,11 +390,12 @@ PrefService* InitializeLocalState(
     if (!local_state_file_exists) {
       base::FilePath parent_profile =
           parsed_command_line.GetSwitchValuePath(switches::kParentProfile);
-      scoped_refptr<PrefRegistrySimple> registry = new PrefRegistrySimple();
+      const auto registry = base::MakeRefCounted<PrefRegistrySimple>();
       std::unique_ptr<PrefService> parent_local_state(
-          chrome_prefs::CreateLocalState(
-              parent_profile, local_state_task_runner,
-              g_browser_process->policy_service(), registry, false, nullptr));
+          chrome_prefs::CreateLocalState(parent_profile,
+                                         local_state_task_runner,
+                                         g_browser_process->policy_service(),
+                                         registry.get(), false, nullptr));
       registry->RegisterStringPref(prefs::kApplicationLocale, std::string());
       // Right now, we only inherit the locale setting from the parent profile.
       local_state->SetString(

chrome/browser/prefs/chrome_pref_service_factory.cc

@@ -440,7 +440,7 @@ std::unique_ptr<PrefService> CreateLocalState(
     const base::FilePath& pref_filename,
     base::SequencedTaskRunner* pref_io_task_runner,
     policy::PolicyService* policy_service,
-    const scoped_refptr<PrefRegistry>& pref_registry,
+    PrefRegistry* pref_registry,
     bool async,
     std::unique_ptr<PrefValueStore::Delegate> delegate) {
   sync_preferences::PrefServiceSyncableFactory factory;
@@ -450,7 +450,7 @@ std::unique_ptr<PrefService> CreateLocalState(
                                    std::unique_ptr<PrefFilter>()),
                  NULL,  // extension_prefs
                  async);
-  return factory.Create(pref_registry.get(), std::move(delegate));
+  return factory.Create(pref_registry, std::move(delegate));
 }
 
 std::unique_ptr<sync_preferences::PrefServiceSyncable> CreateProfilePrefs(
@@ -459,7 +459,7 @@ std::unique_ptr<sync_preferences::PrefServiceSyncable> CreateProfilePrefs(
     policy::PolicyService* policy_service,
     SupervisedUserSettingsService* supervised_user_settings,
     const scoped_refptr<PrefStore>& extension_prefs,
-    const scoped_refptr<user_prefs::PrefRegistrySyncable>& pref_registry,
+    user_prefs::PrefRegistrySyncable* pref_registry,
     bool async,
     scoped_refptr<base::SequencedTaskRunner> io_task_runner,
     std::unique_ptr<PrefValueStore::Delegate> delegate) {
@@ -471,19 +471,16 @@ std::unique_ptr<sync_preferences::PrefServiceSyncable> CreateProfilePrefs(
       base::MakeUnique<ResetOnLoadObserverImpl>(profile_path),
       mojo::MakeRequest(&reset_on_load_observer));
   sync_preferences::PrefServiceSyncableFactory factory;
-  scoped_refptr<PersistentPrefStore> user_pref_store(
+  const scoped_refptr<PersistentPrefStore> user_pref_store =
       CreateProfilePrefStoreManager(profile_path)
           ->CreateProfilePrefStore(
               GetTrackingConfiguration(), kTrackedPrefsReportingIDsCount,
               std::move(io_task_runner), std::move(reset_on_load_observer),
-              std::move(validation_delegate)));
+              std::move(validation_delegate));
   PrepareFactory(&factory, profile_path, policy_service,
                  supervised_user_settings, user_pref_store, extension_prefs,
                  async);
-  std::unique_ptr<sync_preferences::PrefServiceSyncable> pref_service =
-      factory.CreateSyncable(pref_registry.get(), std::move(delegate));
-
-  return pref_service;
+  return factory.CreateSyncable(pref_registry, std::move(delegate));
 }
 
 void DisableDomainCheckForTesting() {

chrome/browser/prefs/chrome_pref_service_factory.h

@@ -68,7 +68,7 @@ std::unique_ptr<PrefService> CreateLocalState(
     const base::FilePath& pref_filename,
     base::SequencedTaskRunner* pref_io_task_runner,
     policy::PolicyService* policy_service,
-    const scoped_refptr<PrefRegistry>& pref_registry,
+    PrefRegistry* pref_registry,
     bool async,
     std::unique_ptr<PrefValueStore::Delegate> delegate);
 
@@ -78,7 +78,7 @@ std::unique_ptr<sync_preferences::PrefServiceSyncable> CreateProfilePrefs(
     policy::PolicyService* policy_service,
     SupervisedUserSettingsService* supervised_user_settings,
     const scoped_refptr<PrefStore>& extension_prefs,
-    const scoped_refptr<user_prefs::PrefRegistrySyncable>& pref_registry,
+    user_prefs::PrefRegistrySyncable* pref_registry,
     bool async,
     scoped_refptr<base::SequencedTaskRunner> io_task_runner,
     std::unique_ptr<PrefValueStore::Delegate> delegate);

chrome/browser/profiles/profile_impl.cc

@@ -393,7 +393,7 @@ ProfileImpl::ProfileImpl(
     scoped_refptr<base::SequencedTaskRunner> io_task_runner)
     : path_(path),
       io_task_runner_(std::move(io_task_runner)),
-      pref_registry_(new user_prefs::PrefRegistrySyncable),
+      pref_registry_(base::MakeRefCounted<user_prefs::PrefRegistrySyncable>()),
       io_data_(this),
       last_session_exit_type_(EXIT_NORMAL),
       start_time_(base::Time::Now()),
@@ -502,8 +502,8 @@ ProfileImpl::ProfileImpl(
     prefs_ = chrome_prefs::CreateProfilePrefs(
         path_, std::move(pref_validation_delegate),
         profile_policy_connector_->policy_service(), supervised_user_settings,
-        CreateExtensionPrefStore(this, false), pref_registry_, async_prefs,
-        GetIOTaskRunner(), std::move(delegate));
+        CreateExtensionPrefStore(this, false), pref_registry_.get(),
+        async_prefs, GetIOTaskRunner(), std::move(delegate));
     // Register on BrowserContext.
     user_prefs::UserPrefs::Set(this, prefs_.get());
   }

chrome/browser/profiles/profile_impl.h

@@ -228,7 +228,7 @@ class ProfileImpl : public Profile {
   // Keep |prefs_| on top for destruction order because |extension_prefs_|,
   // |io_data_| and others store pointers to |prefs_| and shall be destructed
   // first.
-  scoped_refptr<user_prefs::PrefRegistrySyncable> pref_registry_;
+  const scoped_refptr<user_prefs::PrefRegistrySyncable> pref_registry_;
   std::unique_ptr<sync_preferences::PrefServiceSyncable> prefs_;
   // See comment in GetOffTheRecordPrefs. Field exists so something owns the
   // dummy.