Switch Preferences to use Factory

chrome/browser/prefs/preferences_connection_manager.cc

Index: chrome/browser/prefs/preferences_connection_manager.cc
diff --git a/chrome/browser/prefs/preferences_connection_manager.cc b/chrome/browser/prefs/preferences_connection_manager.cc
index 166271307f06b9640d5dbefac9a67394d015468b..10de6ce4e8f64dafa5862eb3069413a22584d171 100644
--- a/chrome/browser/prefs/preferences_connection_manager.cc
+++ b/chrome/browser/prefs/preferences_connection_manager.cc
@@ -47,16 +47,17 @@ void PreferencesConnectionManager::OnConnectionError(
     mojo::StrongBindingPtr<prefs::mojom::PreferencesManager> binding) {
   if (!binding)
     return;
-  for (auto it = std::begin(bindings_); it != std::end(bindings_); ++it) {
+  for (auto it = std::begin(manager_bindings_);

[sky, 2017/01/17 23:45:04]

Why do you need std::begin/std::end here?

[jonross, 2017/01/18 23:13:02]

I need the iterator for std::vector.erase, and ranged-based for loops do not
resolve to that. :(

Nor do we support bound params for lambdas yet, so I can't just use
std::remove_if :(

[sky, 2017/01/18 23:38:25]

Sorry for not being clear. I'm asking why you can't do:

for (auto it = manager_bindings_.begin(); it != manager_bindings_.end(); ++it) {
...
}

Maybe I'm missing something?

[jonross, 2017/01/19 15:24:13]

You aren't missing anything, and now I'm wondering why I wrote it that way to
being with... fixed.

+       it != std::end(manager_bindings_); ++it) {
     if (it->get() == binding.get()) {
-      bindings_.erase(it);
+      manager_bindings_.erase(it);
       return;
     }
   }
 }
 
 void PreferencesConnectionManager::OnProfileDestroyed() {
-  for (auto& it : bindings_) {
+  for (auto& it : manager_bindings_) {
     // Shutdown any PreferenceManager that is still alive.
     if (it)
       it->Close();
@@ -66,8 +67,8 @@ void PreferencesConnectionManager::OnProfileDestroyed() {
 }
 
 void PreferencesConnectionManager::Create(
-    const service_manager::Identity& remote_identity,
-    prefs::mojom::PreferencesManagerRequest request) {
+    prefs::mojom::PreferencesObserverPtr observer,
+    prefs::mojom::PreferencesManagerRequest manager) {
   // Certain tests have no profiles to connect to, and static initializers
   // which block the creation of test profiles.
   if (!g_browser_process->profile_manager()->GetNumberOfProfiles())
@@ -75,13 +76,20 @@ void PreferencesConnectionManager::Create(
 
   Profile* profile = ProfileManager::GetActiveUserProfile();
   mojo::StrongBindingPtr<prefs::mojom::PreferencesManager> binding =
-      mojo::MakeStrongBinding(base::MakeUnique<PreferencesManager>(profile),
-                              std::move(request));
+      mojo::MakeStrongBinding(
+          base::MakeUnique<PreferencesManager>(std::move(observer), profile),
+          std::move(manager));
   // Copying the base::WeakPtr for future deletion.
   binding->set_connection_error_handler(
       base::Bind(&PreferencesConnectionManager::OnConnectionError,
                  base::Unretained(this), binding));
-  bindings_.push_back(std::move(binding));
+  manager_bindings_.push_back(std::move(binding));
+}
+
+void PreferencesConnectionManager::Create(
+    const service_manager::Identity& remote_identity,
+    prefs::mojom::PreferencesFactoryRequest request) {
+  factory_bindings_.AddBinding(this, std::move(request));
 }
 
 void PreferencesConnectionManager::OnStart() {
@@ -101,6 +109,6 @@ void PreferencesConnectionManager::OnStart() {
 bool PreferencesConnectionManager::OnConnect(
     const service_manager::ServiceInfo& remote_info,
     service_manager::InterfaceRegistry* registry) {
-  registry->AddInterface<prefs::mojom::PreferencesManager>(this);
+  registry->AddInterface<prefs::mojom::PreferencesFactory>(this);
   return true;
 }