chromeos: Add power supply info reading capability

chrome/browser/chromeos/chrome_browser_main_chromeos.cc

 // 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.
 
 #include "chrome/browser/chromeos/chrome_browser_main_chromeos.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/command_line.h"
 #include "base/lazy_instance.h"
 #include "base/message_loop.h"
 #include "chrome/browser/chromeos/boot_times_loader.h"
+#include "chrome/browser/chromeos/brightness_observer.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
+#include "chrome/browser/chromeos/cros/power_library.h"
 #include "chrome/browser/chromeos/dbus/dbus_thread_manager.h"
 #include "chrome/browser/chromeos/net/cros_network_change_notifier_factory.h"
 #include "chrome/browser/chromeos/system/statistics_provider.h"
 #include "chrome/browser/defaults.h"
 #include "chrome/common/chrome_switches.h"
 #include "content/common/main_function_params.h"
 #include "net/base/network_change_notifier.h"
 
 #if defined(TOOLKIT_USES_GTK)
 #include <gtk/gtk.h>
@@ skipping 40 matching lines @@
 };
 
 static base::LazyInstance<MessageLoopObserver> g_message_loop_observer(
     base::LINKER_INITIALIZED);
 
 ChromeBrowserMainPartsChromeos::ChromeBrowserMainPartsChromeos(
     const MainFunctionParams& parameters)
     : ChromeBrowserMainPartsGtk(parameters) {
 }
 
 ChromeBrowserMainPartsChromeos::~ChromeBrowserMainPartsChromeos() {

[satorux1, 2011/10/14 05:56:28]

Just like http://codereview.chromium.org/8271010/, please remove the observer
from >power_manager_client() like:

chromeos::DBusThreadManager::Get()->power_manager_client()->
    RemoveObserver(power_lib);

[Simon Que, 2011/10/18 19:23:06]

Done.

   chromeos::DBusThreadManager::Shutdown();
 
   if (!parameters().ui_task && chromeos::CrosLibrary::Get())
     chromeos::CrosLibrary::Shutdown();
 
   // To be precise, logout (browser shutdown) is not yet done, but the
   // remaining work is negligible, hence we say LogoutDone here.
   chromeos::BootTimesLoader::Get()->AddLogoutTimeMarker("LogoutDone",
                                                         false);
   chromeos::BootTimesLoader::Get()->WriteLogoutTimes();
@@ skipping 33 matching lines @@
 }
 
 void ChromeBrowserMainPartsChromeos::PostMainMessageLoopStart() {
   ChromeBrowserMainPartsPosix::PostMainMessageLoopStart();
   MessageLoopForUI* message_loop = MessageLoopForUI::current();
   message_loop->AddObserver(g_message_loop_observer.Pointer());
 
   // Initialize DBusThreadManager for the browser. This must be done after
   // the main message loop is started, as it uses the message loop.
   chromeos::DBusThreadManager::Initialize();
+
+  // Initialize the brightness observer so that we'll display an onscreen
+  // indication of brightness changes during login.
+  static chromeos::BrightnessObserver* brightness_observer =
+      new chromeos::BrightnessObserver();
+  chromeos::DBusThreadManager::Get()->power_manager_client()->AddObserver(
+      brightness_observer);

[satorux1, 2011/10/14 05:56:28]

Sorry, I just submitted a patch to clean this up:
http://codereview.chromium.org/8271010/

Please sync your client and fix conflicts carefully.

[Simon Que, 2011/10/18 19:23:06]

Done.

+
+  // Probe for power supply status.
+  static chromeos::PowerManagerClient::Observer* power_lib =

[satorux1, 2011/10/14 05:56:28]

Please remove static.

[Simon Que, 2011/10/18 19:23:06]

Done.

+      chromeos::CrosLibrary::Get()->GetPowerLibrary();
+  chromeos::DBusThreadManager::Get()->power_manager_client()->
+      AddObserver(power_lib);
 }