Add device policy to disallow shutdown - ash UI modifications

ash/system/tray/system_tray_delegate.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef ASH_SYSTEM_TRAY_SYSTEM_TRAY_DELEGATE_H_
 #define ASH_SYSTEM_TRAY_SYSTEM_TRAY_DELEGATE_H_
 
 #include <string>
 #include <vector>
 
 #include "ash/ash_export.h"
 #include "ash/system/user/login_status.h"
 #include "base/callback_forward.h"
 #include "base/files/file_path.h"
 #include "base/i18n/time_formatting.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string16.h"
 #include "ui/gfx/image/image_skia.h"
 
 namespace base {
 class TimeDelta;
 class TimeTicks;
 }
 
 namespace ash {
 
 class CustodianInfoTrayObserver;
+class ShutdownPolicyObserver;
 
 struct ASH_EXPORT NetworkIconInfo {
   NetworkIconInfo();
   ~NetworkIconInfo();
 
   bool highlight() const { return connected || connecting; }
 
   bool connecting;
   bool connected;
   bool tray_icon_visible;
@@ skipping 53 matching lines @@
 
   UpdateSeverity severity;
   bool update_required;
   bool factory_reset_required;
 };
 
 typedef std::vector<IMEInfo> IMEInfoList;
 
 class VolumeControlDelegate;
 
+using RebootOnShutdownCallback = base::Callback<void(bool)>;

[stevenjb, 2014/12/17 19:09:02]

Update other typedefs in this header to 'using'

[cschuet (SLOW), 2015/01/07 14:37:07]

Done.

+
 namespace tray {
 class UserAccountsDelegate;
 }  // namespace tray
 
 class ASH_EXPORT SystemTrayDelegate {
  public:
   virtual ~SystemTrayDelegate() {}
 
   // Called after SystemTray has been instantiated.
   virtual void Initialize() = 0;
@@ skipping 81 matching lines @@
 
   // Shows information about enterprise enrolled devices.
   virtual void ShowEnterpriseInfo() = 0;
 
   // Shows information about supervised users.
   virtual void ShowSupervisedUserInfo() = 0;
 
   // Shows login UI to add other users to this session.
   virtual void ShowUserLogin() = 0;
 
+  // Attempts to reboot the system.

[stevenjb, 2014/12/17 19:09:02]

restart

[cschuet (SLOW), 2015/01/07 14:37:07]

This function has been removed in the new version of this CL.

+  virtual void Restart() = 0;
+
   // Attempts to shut down the system.
   virtual void ShutDown() = 0;
 
   // Attempts to sign out the user.
   virtual void SignOut() = 0;
 
   // Attempts to lock the screen.
   virtual void RequestLockScreen() = 0;
 
   // Attempts to restart the system for update.
@@ skipping 78 matching lines @@
   // Returns accounts delegate for given user.
   virtual tray::UserAccountsDelegate* GetUserAccountsDelegate(
       const std::string& user_id) = 0;
 
   // Adding observers that are notified when supervised info is being changed.
   virtual void AddCustodianInfoTrayObserver(
       CustodianInfoTrayObserver* observer) = 0;
 
   virtual void RemoveCustodianInfoTrayObserver(
       CustodianInfoTrayObserver* observer) = 0;
+
+  // Adds an observer whose |OnShutdownPolicyChanged| function is called when
+  // the |DeviceRebootOnShutdown| policy changes. If this policy is set to
+  // true, a device cannot be shut down anymore but only rebooted.
+  virtual void AddShutdownPolicyObserver(ShutdownPolicyObserver* observer) = 0;
+
+  // Determines whether the device is automatically rebooted when shut down as
+  // specified by the device policy |DeviceRebootOnShutdown|. This function
+  // asynchronously calls |RebootOnShutdownCallback| once a trusted policy

[stevenjb, 2014/12/17 19:09:02]

Use variable name |callback| instead of type.

[cschuet (SLOW), 2015/01/07 14:37:07]

Done.

+  // becomes available.
+  virtual void ShouldRebootOnShutdown(
+      const RebootOnShutdownCallback& callback) = 0;

[stevenjb, 2014/12/17 19:09:02]

This should be named something like 'RequestShouldRebootOnShutdown', otherwise
it looks like a setter.

[cschuet (SLOW), 2015/01/07 14:37:07]

ShouldRebootOnShutdown is supposed to sound like a question "Should we reboot on
shutdown?". I think a setter would be called SetRebootOnShutdown. If you feel
strongly about this I will change it, but 'RequestShouldRebootOnShutdown' is a
bit clunky, I think.

+
+  virtual void RemoveShutdownPolicyObserver(

[stevenjb, 2014/12/17 19:09:02]

Remove should follow Add here and elsewhere.

[cschuet (SLOW), 2015/01/07 14:37:07]

Done.

+      ShutdownPolicyObserver* observer) = 0;
 };
 
 }  // namespace ash
 
 #endif  // ASH_SYSTEM_TRAY_SYSTEM_TRAY_DELEGATE_H_