Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3)

Unified Diff: chrome/browser/prefs/preferences_connection_manager.cc

Issue 2778733003: Pref service: delete old implementation (Closed)
Patch Set: Fix missed merge Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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
deleted file mode 100644
index c2ca921465903a24a17f5e3ed7acca671b255943..0000000000000000000000000000000000000000
--- a/chrome/browser/prefs/preferences_connection_manager.cc
+++ /dev/null
@@ -1,86 +0,0 @@
-// 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_connection_manager.h"
-
-#include "base/memory/ptr_util.h"
-#include "chrome/browser/browser_process.h"
-#include "chrome/browser/prefs/preferences_service.h"
-#include "chrome/browser/profiles/profile.h"
-#include "chrome/browser/profiles/profile_manager.h"
-#include "chrome/browser/signin/profile_oauth2_token_service_factory.h"
-#include "components/keyed_service/content/browser_context_keyed_service_shutdown_notifier_factory.h"
-#include "services/service_manager/public/cpp/interface_registry.h"
-
-namespace {
-
-// Required singleton subclass to control dependencies between
-// PreferenceConnectionManager and the KeyedServiceShutdownNotifier.
-class ShutdownNotifierFactory
- : public BrowserContextKeyedServiceShutdownNotifierFactory {
- public:
- static ShutdownNotifierFactory* GetInstance() {
- return base::Singleton<ShutdownNotifierFactory>::get();
- }
-
- private:
- friend struct base::DefaultSingletonTraits<ShutdownNotifierFactory>;
-
- ShutdownNotifierFactory()
- : BrowserContextKeyedServiceShutdownNotifierFactory(
- "PreferencesConnectionManager") {
- DependsOn(ProfileOAuth2TokenServiceFactory::GetInstance());
- }
- ~ShutdownNotifierFactory() override {}
-
- DISALLOW_COPY_AND_ASSIGN(ShutdownNotifierFactory);
-};
-
-} // namespace
-
-PreferencesConnectionManager::PreferencesConnectionManager() {}
-
-PreferencesConnectionManager::~PreferencesConnectionManager() {}
-
-void PreferencesConnectionManager::OnProfileDestroyed() {
- // Shutdown any PreferenceService that is still alive.
- manager_bindings_.CloseAllBindings();
- profile_shutdown_notification_.reset();
-}
-
-void PreferencesConnectionManager::Create(
- prefs::mojom::PreferencesServiceClientPtr client,
- prefs::mojom::PreferencesServiceRequest service) {
- // 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())
- return;
-
- manager_bindings_.AddBinding(
- base::MakeUnique<PreferencesService>(
- std::move(client), ProfileManager::GetActiveUserProfile()),
- std::move(service));
-}
-
-void PreferencesConnectionManager::Create(
- const service_manager::Identity& remote_identity,
- prefs::mojom::PreferencesServiceFactoryRequest request) {
- factory_bindings_.AddBinding(this, std::move(request));
-
- if (!profile_shutdown_notification_) {
- profile_shutdown_notification_ =
- ShutdownNotifierFactory::GetInstance()
- ->Get(ProfileManager::GetActiveUserProfile())
- ->Subscribe(
- base::Bind(&PreferencesConnectionManager::OnProfileDestroyed,
- base::Unretained(this)));
- }
-}
-
-bool PreferencesConnectionManager::OnConnect(
- const service_manager::ServiceInfo& remote_info,
- service_manager::InterfaceRegistry* registry) {
- registry->AddInterface<prefs::mojom::PreferencesServiceFactory>(this);
- return true;
-}
« no previous file with comments | « chrome/browser/prefs/preferences_connection_manager.h ('k') | chrome/browser/prefs/preferences_manifest.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698