aura: brightness and volume bubble.

chrome/browser/chromeos/volume_bubble.h

Index: chrome/browser/chromeos/volume_bubble.h
diff --git a/chrome/browser/chromeos/volume_bubble.h b/chrome/browser/chromeos/volume_bubble.h
index 17f72fc1a19190531045f546058336a2b61af3e0..9bcfe56bd9b991d818d985b1a32be2dde89542ae 100644
--- a/chrome/browser/chromeos/volume_bubble.h
+++ b/chrome/browser/chromeos/volume_bubble.h
@@ -11,18 +11,32 @@
 
 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 volume bubble.
-class VolumeBubble : public SettingLevelBubble {
+class VolumeBubble {
  public:
+  // Returns a the singleton volume bubble instance.

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

ditto here on dan's comment elsewhere; remove "a"

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

Done.

   static VolumeBubble* 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);
+
+  // hides the bubble.

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

capitalize 'hides'

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

Done.

+  void HideBubble();
+
  private:
   friend struct DefaultSingletonTraits<VolumeBubble>;
 
+  // Bubble widget, owned by VolumeBubble.
+  views::Widget* widget_;
   VolumeBubble();
-  virtual ~VolumeBubble() {}
+  virtual ~VolumeBubble();
 
   DISALLOW_COPY_AND_ASSIGN(VolumeBubble);
 };