Switch Preferences to use Factory

chrome/browser/prefs/preferences_manager_unittest.cc

 // Copyright 2016 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/preferences_manager.h"
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 144 matching lines @@
   registry_ = service->registry();
   chrome::RegisterUserProfilePrefs(registry_);
 
   const std::string kName = "navi";
   profile_ = testing_profile_manager_.CreateTestingProfile(
       kName, std::move(service), base::UTF8ToUTF16(kName), 0, std::string(),
       TestingProfile::TestingFactories());
   ASSERT_NE(nullptr, profile_->GetPrefs());
 
   observer_.reset(new TestPreferencesObserver(mojo::MakeRequest(&proxy_)));
-  manager_ = base::MakeUnique<PreferencesManager>(profile_);
+  manager_ = base::MakeUnique<PreferencesManager>(std::move(proxy_), profile_);
   ASSERT_TRUE(manager_->preferences_change_registrar_->IsEmpty());
-  manager_->AddObserver(std::move(proxy_));
 }
 
 void PreferencesManagerTest::TearDown() {
   testing_profile_manager_.DeleteAllTestingProfiles();
 }
 
 // Tests that when the PrefService is empty that no subscriptions are made.
 TEST_F(PreferencesManagerTest, EmptyService) {
   const std::string kKey = "hey";
   std::vector<std::string> preferences;
@@ skipping 120 matching lines @@
   std::unique_ptr<base::DictionaryValue> dictionary =
       base::MakeUnique<base::DictionaryValue>();
   dictionary->SetInteger(kKey, kNewValue);
   SetPreferences(std::move(dictionary));
 
   EXPECT_FALSE(observer()->on_preferences_changed_called());
   EXPECT_EQ(kNewValue, service()->GetInteger(kKey));
 }
 
 }  // namespace test