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

Unified Diff: chrome/browser/chromeos/chrome_browser_main_chromeos.cc

Issue 10832035: Switch from SignedSettings to DeviceSettingsService. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Created 8 years, 4 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/chromeos/chrome_browser_main_chromeos.cc
diff --git a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
index 2fa9ca8a253d5aae8ef41ae0c935dc4e5626dc2f..8fcf8ee3c3d82148e8c283f16238c25050a2f60f 100644
--- a/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
+++ b/chrome/browser/chromeos/chrome_browser_main_chromeos.cc
@@ -51,8 +51,8 @@
#include "chrome/browser/chromeos/power/screen_lock_observer.h"
#include "chrome/browser/chromeos/power/user_activity_notifier.h"
#include "chrome/browser/chromeos/power/video_activity_notifier.h"
-#include "chrome/browser/chromeos/settings/ownership_service.h"
-#include "chrome/browser/chromeos/settings/session_manager_observer.h"
+#include "chrome/browser/chromeos/settings/device_settings_service.h"
+#include "chrome/browser/chromeos/settings/owner_key_util.h"
#include "chrome/browser/chromeos/system/statistics_provider.h"
#include "chrome/browser/chromeos/system_key_event_listener.h"
#include "chrome/browser/chromeos/upgrade_detector_chromeos.h"
@@ -302,9 +302,11 @@ void ChromeBrowserMainPartsChromeos::PostMainMessageLoopStart() {
chromeos::CrosDBusService::Initialize();
- // Initialize the session manager observer so that we'll take actions
- // per signals sent from the session manager.
- session_manager_observer_.reset(new chromeos::SessionManagerObserver);
+ // Initialize the device settings service so that we'll take actions per
+ // signals sent from the session manager.
+ chromeos::DeviceSettingsService::Get()->Initialize(
+ chromeos::DBusThreadManager::Get()->GetSessionManagerClient(),
+ chromeos::OwnerKeyUtil::Create());
chromeos::disks::DiskMountManager::Initialize();
chromeos::mtp::MediaTransferProtocolManager::Initialize();
@@ -357,7 +359,7 @@ void ChromeBrowserMainPartsChromeos::PreProfileInit() {
chromeos::BootTimesLoader::Get()->RecordChromeMainStats();
// Trigger prefetching of ownership status.
- chromeos::OwnershipService::GetSharedInstance()->Prewarm();
+ chromeos::DeviceSettingsService::Get()->Load();
// -- This used to be in ChromeBrowserMainParts::PreMainMessageLoopRun()
// -- just before CreateProfile().
@@ -528,9 +530,11 @@ void ChromeBrowserMainPartsChromeos::PostMainMessageLoopRun() {
if (chromeos::CrosNetworkChangeNotifierFactory::GetInstance())
chromeos::CrosNetworkChangeNotifierFactory::GetInstance()->Shutdown();
+ // Tell DeviceSettingsService to stop talking to session_manager.
+ chromeos::DeviceSettingsService::Get()->Shutdown();
+
// We should remove observers attached to D-Bus clients before
// DBusThreadManager is shut down.
- session_manager_observer_.reset();
screen_lock_observer_.reset();
resume_observer_.reset();
brightness_observer_.reset();

Powered by Google App Engine
This is Rietveld 408576698