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

Unified Diff: chrome/browser/chromeos/accessibility/magnification_manager.cc

Issue 2661283002: cros: Clean up SessionStateDelegate refs in Chrome (Closed)
Patch Set: update browser_finder_chromeos_unittest Created 3 years, 10 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/accessibility/magnification_manager.cc
diff --git a/chrome/browser/chromeos/accessibility/magnification_manager.cc b/chrome/browser/chromeos/accessibility/magnification_manager.cc
index 388715d9fa8da2f158e32b37b66934f08446942f..60b53ba76b8428319dedbcc159b27e60f69565fe 100644
--- a/chrome/browser/chromeos/accessibility/magnification_manager.cc
+++ b/chrome/browser/chromeos/accessibility/magnification_manager.cc
@@ -8,7 +8,6 @@
#include <memory>
#include "ash/common/accessibility_types.h"
-#include "ash/common/session/session_state_observer.h"
#include "ash/magnifier/magnification_controller.h"
#include "ash/magnifier/partial_magnification_controller.h"
#include "ash/shell.h"
@@ -22,6 +21,7 @@
#include "chrome/common/pref_names.h"
#include "components/prefs/pref_member.h"
#include "components/prefs/pref_service.h"
+#include "components/user_manager/user_manager.h"
#include "content/public/browser/focused_node_details.h"
#include "content/public/browser/notification_details.h"
#include "content/public/browser/notification_observer.h"
@@ -29,17 +29,16 @@
#include "content/public/browser/notification_service.h"
#include "content/public/browser/notification_source.h"
-class AccountId;
-
namespace chromeos {
namespace {
static MagnificationManager* g_magnification_manager = NULL;
}
-class MagnificationManagerImpl : public MagnificationManager,
- public content::NotificationObserver,
- public ash::SessionStateObserver {
+class MagnificationManagerImpl
+ : public MagnificationManager,
+ public content::NotificationObserver,
+ public user_manager::UserManager::UserSessionStateObserver {
public:
MagnificationManagerImpl()
: profile_(NULL),
@@ -108,8 +107,8 @@ class MagnificationManagerImpl : public MagnificationManager,
void SetProfileForTest(Profile* profile) override { SetProfile(profile); }
- // SessionStateObserver overrides:
- void ActiveUserChanged(const AccountId& account_id) override {
+ // user_manager::UserManager::UserSessionStateObserver overrides:
+ void ActiveUserChanged(const user_manager::User* active_user) override {
SetProfile(ProfileManager::GetActiveUserProfile());
}
@@ -258,9 +257,9 @@ class MagnificationManagerImpl : public MagnificationManager,
SetProfile(ProfileManager::GetActiveUserProfile());
// Add a session state observer to be able to monitor session changes.
- if (!session_state_observer_.get() && ash::Shell::HasInstance())
+ if (!session_state_observer_.get())
session_state_observer_.reset(
- new ash::ScopedSessionStateObserver(this));
+ new user_manager::ScopedUserSessionStateObserver(this));
break;
case chrome::NOTIFICATION_PROFILE_DESTROYED: {
// Update |profile_| when exiting a session or shutting down.
@@ -294,7 +293,8 @@ class MagnificationManagerImpl : public MagnificationManager,
content::NotificationRegistrar registrar_;
std::unique_ptr<PrefChangeRegistrar> pref_change_registrar_;
- std::unique_ptr<ash::ScopedSessionStateObserver> session_state_observer_;
+ std::unique_ptr<user_manager::ScopedUserSessionStateObserver>
+ session_state_observer_;
DISALLOW_COPY_AND_ASSIGN(MagnificationManagerImpl);
};

Powered by Google App Engine
This is Rietveld 408576698