chromeos: Simplify power supply info in PowerLibrary

chrome/browser/automation/testing_automation_provider.h

 // Copyright (c) 2011 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 CHROME_BROWSER_AUTOMATION_TESTING_AUTOMATION_PROVIDER_H_
 #define CHROME_BROWSER_AUTOMATION_TESTING_AUTOMATION_PROVIDER_H_
 #pragma once
 
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "chrome/browser/automation/automation_provider.h"
 #include "chrome/browser/automation/automation_provider_json.h"
 #include "chrome/browser/history/history.h"
 #include "chrome/browser/importer/importer_list_observer.h"
 #include "chrome/browser/sync/profile_sync_service_harness.h"
 #include "chrome/browser/ui/browser_list.h"
 #include "content/public/browser/notification_registrar.h"
 #include "content/public/common/page_type.h"
 #include "net/base/cert_status_flags.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebInputEvent.h"
 
+#if defined(OS_CHROMEOS)
+#include "chrome/browser/chromeos/cros/power_library.h"
+#endif  // defined(OS_CHROMEOS)
+
 class ImporterList;
 class TemplateURLService;
 
 namespace base {
 class DictionaryValue;
 }
 
 namespace webkit {
 struct WebPluginInfo;
 }
 
 // This is an automation provider containing testing calls.
 class TestingAutomationProvider : public AutomationProvider,
                                   public BrowserList::Observer,
                                   public importer::ImporterListObserver,
+#if defined(OS_CHROMEOS)
+                                  // TODO(sque): this is less than ideal.  It
+                                  // should be in a separate class defined in a
+                                  // source file.

[stevenjb, 2011/10/25 21:41:49]

nit: I created crosbug.com/22081, we should make this:
// TODO(sque): Move this to a ChromeOS specific subclass. See crosbug.com/22081

Also, I'd put the comment with the first #define instead (plus it will fit on
one line there).

[Simon Que, 2011/10/25 22:59:16]

Done.

+                                  public chromeos::PowerLibrary::Observer,
+#endif // defined(OS_CHROMEOS)
                                   public content::NotificationObserver {
+
  public:
   explicit TestingAutomationProvider(Profile* profile);
 
   virtual IPC::Channel::Mode GetChannelMode(bool use_named_interface);
 
   // IPC::Channel::Listener:
   virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE;
   virtual void OnChannelError() OVERRIDE;
 
  private:
@@ skipping 1285 matching lines @@
   // Volume.
   void GetVolumeInfo(base::DictionaryValue* args, IPC::Message* reply_message);
 
   void SetVolume(base::DictionaryValue* args, IPC::Message* reply_message);
 
   void SetMute(base::DictionaryValue* args, IPC::Message* reply_message);
 
   void CaptureProfilePhoto(Browser* browser,
                            DictionaryValue* args,
                            IPC::Message* reply_message);
+
+  // chromeos::PowerLibrary::Observer overrides.
+  virtual void PowerChanged(const chromeos::PowerSupplyStatus& status) OVERRIDE;
+  virtual void SystemResumed() OVERRIDE {}
 #endif  // defined(OS_CHROMEOS)
 
   void WaitForTabCountToBecome(int browser_handle,
                                int target_tab_count,
                                IPC::Message* reply_message);
 
   void WaitForInfoBarCount(int tab_handle,
                            size_t target_count,
                            IPC::Message* reply_message);
 
@@ skipping 52 matching lines @@
   // Used to enumerate browser profiles.
   scoped_refptr<ImporterList> importer_list_;
 
   // The stored data for the ImportSettings operation.
   ImportSettingsData import_settings_data_;
 
   DISALLOW_COPY_AND_ASSIGN(TestingAutomationProvider);
 };
 
 #endif  // CHROME_BROWSER_AUTOMATION_TESTING_AUTOMATION_PROVIDER_H_