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

Unified Diff: ash/system/tray/system_tray_delegate.h

Issue 2832903002: cros: Remove supervised user methods from SystemTrayDelegate (Closed)
Patch Set: rebase Created 3 years, 8 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
« no previous file with comments | « ash/system/tray/system_tray.cc ('k') | ash/system/tray/system_tray_delegate.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ash/system/tray/system_tray_delegate.h
diff --git a/ash/system/tray/system_tray_delegate.h b/ash/system/tray/system_tray_delegate.h
index 5056e5709ee2905999c95961c4f51a6f08734821..1f66cf6ca18930f23576f7c15c29fdae4b080749 100644
--- a/ash/system/tray/system_tray_delegate.h
+++ b/ash/system/tray/system_tray_delegate.h
@@ -25,7 +25,6 @@ namespace ash {
struct IMEInfo;
struct IMEPropertyInfo;
-class CustodianInfoTrayObserver;
class SystemTray;
class SystemTrayItem;
@@ -37,15 +36,10 @@ class NetworkingConfigDelegate;
// SystemTrayDelegate is intended for delegating tasks in the System Tray to the
// application (e.g. Chrome). These tasks should be limited to application
// (browser) specific tasks. For non application specific tasks, where possible,
-// components/, chromeos/, device/, etc., code should be used directly. If more
-// than one related method is being added, consider adding an additional
-// specific delegate (e.g. CastConfigDelegate).
+// components/, chromeos/, device/, etc., code should be used directly.
//
-// These methods should all have trivial default implementations for platforms
-// that do not implement the method (e.g. return false or nullptr). This
-// eliminates the need to propagate default implementations across the various
-// implementations of this class. Consumers of this delegate should handle the
-// default return value (e.g. nullptr).
+// DEPRECATED: This class is being replaced with SystemTrayController and
+// SessionController to support mash/mustash. Add new code to those classes.
class ASH_EXPORT SystemTrayDelegate {
public:
SystemTrayDelegate();
@@ -67,19 +61,6 @@ class ASH_EXPORT SystemTrayDelegate {
// Returns notification for enterprise enrolled devices.
virtual base::string16 GetEnterpriseMessage() const;
- // Returns the display email of the user that manages the current supervised
- // user.
- // TODO(jamescook): Migrate to SessionController. http://crbug.com/712799
- virtual std::string GetSupervisedUserManager() const;
-
- // Returns the name of the user that manages the current supervised user.
- // TODO(jamescook): Migrate to SessionController. http://crbug.com/712799
- virtual base::string16 GetSupervisedUserManagerName() const;
-
- // Returns the notification for supervised users.
- // TODO(jamescook): Migrate to SessionController. http://crbug.com/712799
- virtual base::string16 GetSupervisedUserMessage() const;
-
// Shows information about enterprise enrolled devices.
virtual void ShowEnterpriseInfo();
@@ -121,13 +102,6 @@ class ASH_EXPORT SystemTrayDelegate {
// Returns true when the Search key is configured to be treated as Caps Lock.
virtual bool IsSearchKeyMappedToCapsLock();
- // Adding observers that are notified when supervised info is being changed.
- virtual void AddCustodianInfoTrayObserver(
- CustodianInfoTrayObserver* observer);
-
- virtual void RemoveCustodianInfoTrayObserver(
- CustodianInfoTrayObserver* observer);
-
// Creates a system tray item for display rotation lock.
// TODO(jamescook): Remove this when mus has support for display management
// and we have a DisplayManager equivalent. See http://crbug.com/548429
« no previous file with comments | « ash/system/tray/system_tray.cc ('k') | ash/system/tray/system_tray_delegate.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698