aura: brightness and volume bubble.

chrome/browser/chromeos/brightness_bubble.h

 // Copyright (c) 2010 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_CHROMEOS_BRIGHTNESS_BUBBLE_H_
 #define CHROME_BROWSER_CHROMEOS_BRIGHTNESS_BUBBLE_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "chrome/browser/chromeos/setting_level_bubble.h"
 
 template <typename T> struct DefaultSingletonTraits;
 
+namespace views {
+class Widget;

[msw, 2011/10/20 19:36:13]

forward decl shouldn't be needed.

[alicet1, 2011/10/20 22:36:30]

Done.

+}
+
 namespace chromeos {
 
 // Singleton class controlling brightness bubble.
-class BrightnessBubble : public SettingLevelBubble {
+class BrightnessBubble {
  public:
+  // Returns a the singleton brightness bubble instance.

[Daniel Erat, 2011/10/20 15:43:39]

nit: delete "a"

[alicet1, 2011/10/20 22:36:30]

Done.

   static BrightnessBubble* GetInstance();
 
+  // Updates the target |percent| and icon to be used, sets
+  // the bubble to |enabled| and shows the bubble.
+  void ShowBubble(double percent, bool enabled);
+
+  // Update the bubble |level| and |enabled| state. See

[Daniel Erat, 2011/10/20 15:43:39]

tiny nit: s/Update/Updates/; add trailing period to second sentence

[alicet1, 2011/10/20 22:36:30]

Done.

+  // comments at setting_level_bubble.h
+  void UpdateWithoutShowingBubble(int level, bool enabled);
+
+  // Hides the bubble.
+  void HideBubble();
+
  private:
   friend struct DefaultSingletonTraits<BrightnessBubble>;
 
+  // Bubble widget, owned by BrightnessBubble.
+  views::Widget* widget_;
+
   BrightnessBubble();
-  virtual ~BrightnessBubble() {}
+  virtual ~BrightnessBubble();
 
   DISALLOW_COPY_AND_ASSIGN(BrightnessBubble);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_CHROMEOS_BRIGHTNESS_BUBBLE_H_