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

Unified Diff: components/user_manager/user_manager.h

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: components/user_manager/user_manager.h
diff --git a/components/user_manager/user_manager.h b/components/user_manager/user_manager.h
index fbf4cc185bf66865199b23ff1c50a531d5b03759..1029959282f0755e1e043a784c87ac2a2172d9d0 100644
--- a/components/user_manager/user_manager.h
+++ b/components/user_manager/user_manager.h
@@ -55,7 +55,7 @@ class USER_MANAGER_EXPORT UserManager {
virtual ~Observer();
};
- // TODO(nkostylev): Refactor and move this observer out of UserManager.
+ // TODO(xiyuan): Refactor and move this observer out of UserManager.
// Observer interface that defines methods used to notify on user session /
// active user state changes. Default implementation is empty.
class UserSessionStateObserver {
@@ -395,6 +395,19 @@ class USER_MANAGER_EXPORT UserManager {
static UserManager* SetForTesting(UserManager* user_manager);
};
+// TODO(xiyuan): Move this along with UserSessionStateObserver
+class USER_MANAGER_EXPORT ScopedUserSessionStateObserver {
+ public:
+ explicit ScopedUserSessionStateObserver(
+ UserManager::UserSessionStateObserver* observer);
+ ~ScopedUserSessionStateObserver();
+
+ private:
+ UserManager::UserSessionStateObserver* const observer_;
+
+ DISALLOW_COPY_AND_ASSIGN(ScopedUserSessionStateObserver);
+};
+
} // namespace user_manager
#endif // COMPONENTS_USER_MANAGER_USER_MANAGER_H_
« no previous file with comments | « chrome/browser/ui/webui/options/chromeos/core_chromeos_options_handler.cc ('k') | components/user_manager/user_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698