chromeos: Simplify ShutdownPolicyHandler

chrome/browser/chromeos/settings/shutdown_policy_handler.h

Index: chrome/browser/chromeos/settings/shutdown_policy_handler.h
diff --git a/chrome/browser/chromeos/settings/shutdown_policy_handler.h b/chrome/browser/chromeos/settings/shutdown_policy_handler.h
index a13dfe9bf6c74de7b2d242a779b2962127f8ef86..30531e169ee4229442dd535534e35262cc5e2459 100644
--- a/chrome/browser/chromeos/settings/shutdown_policy_handler.h
+++ b/chrome/browser/chromeos/settings/shutdown_policy_handler.h
@@ -7,7 +7,6 @@
 
 #include <memory>
 
-#include "base/callback_forward.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
 #include "chrome/browser/chromeos/settings/cros_settings.h"
@@ -19,11 +18,6 @@ namespace chromeos {
 // calling its OnShutdownPolicyChanged method with the new state of the policy.
 class ShutdownPolicyHandler {
  public:
-  // This callback is passed to CheckIfRebootOnShutdown, which invokes it with
-  // the current state of the |DeviceRebootOnShutdown| policy once a trusted of
-  // policies is established.
-  using RebootOnShutdownCallback = base::Callback<void(bool)>;
-
   // This delegate is called when the |DeviceRebootOnShutdown| policy changes.
   class Delegate {
    public:
@@ -36,18 +30,11 @@ class ShutdownPolicyHandler {
   ShutdownPolicyHandler(CrosSettings* cros_settings, Delegate* delegate);
   ~ShutdownPolicyHandler();
 
-  // Once a trusted set of policies is established, this function calls
-  // |callback| with the trusted state of the |DeviceRebootOnShutdown| policy.
-  void CheckIfRebootOnShutdown(const RebootOnShutdownCallback& callback);
-
-  // Resets the policy subscription and clears the delegate.
-  void Shutdown();
+  // Once a trusted set of policies is established, this function notifies the
+  // |delegate_| with the trusted state of the |DeviceRebootOnShutdown| policy.
+  void CheckIfRebootOnShutdown();

[Daniel Erat, 2016/11/11 00:13:00]

just thinking aloud here, but would a name like
NotifyDelegateIfShutdownPolicyChanged be clearer? "check" feels a bit ambiguous.
i sometimes name methods "check" when they CHECK() something. :-P

 
  private:
-  void CallDelegate(bool reboot_on_shutdown);
-
-  void OnShutdownPolicyChanged();
-
   CrosSettings* cros_settings_;
 
   Delegate* delegate_;