Add GetScreenMode() to status_area_host.h.

Add GetScreenMode() to status_area_host.h, and remove IsBrowserMode() and IsScreenLockerMode() APIs. The old APIs seem to be a bit confusing since only 3 combinations out of 2^2 are valid.  (I mean, IsBrowserMode() == true and IsScreenLockerMode() == true is invalid.)

BUG=chromium-os:8386
TEST=manually, try bot.

Review URL: http://codereview.chromium.org/6378008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@72120 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 9 insertions, 5 deletions

diff --git a/chrome/browser/chromeos/status/status_area_host.h b/chrome/browser/chromeos/status/status_area_host.h
index 7b7691e..e7e81af3 100644
--- a/chrome/browser/chromeos/status/status_area_host.h
+++ b/chrome/browser/chromeos/status/status_area_host.h
@@ -38,11 +38,15 @@ class StatusAreaHost {
   // Executes browser command.
   virtual void ExecuteBrowserCommand(int id) const = 0;
 
-  // True if status area hosted in browser. Otherwise it's OOBE/login state.
-  virtual bool IsBrowserMode() const = 0;
-
-  // True if status area hosted in screen locker.
-  virtual bool IsScreenLockerMode() const = 0;
+  // The type of screen the host window is on.
+  enum ScreenMode {
+    kLoginMode,  // The host is for the OOBE/login screens.
+    kBrowserMode,  // The host is for browser.
+    kScreenLockerMode,  // The host is for screen locker.
+  };
+
+  // Returns the type of screen.
+  virtual ScreenMode GetScreenMode() const = 0;
 
  protected:
   virtual ~StatusAreaHost() {}