aura: brightness and volume bubble.

chrome/browser/chromeos/brightness_bubble.h

Index: chrome/browser/chromeos/brightness_bubble.h
diff --git a/chrome/browser/chromeos/brightness_bubble.h b/chrome/browser/chromeos/brightness_bubble.h
index 7b5ed8c5cf9a47d2b7d197fba0829fb24dda93e3..5ca2ee6c5eec7b11995e5040cf62f355611f7c04 100644
--- a/chrome/browser/chromeos/brightness_bubble.h
+++ b/chrome/browser/chromeos/brightness_bubble.h
@@ -14,15 +14,30 @@ template <typename T> struct DefaultSingletonTraits;
 namespace chromeos {
 
 // Singleton class controlling brightness bubble.
-class BrightnessBubble : public SettingLevelBubble {
+class BrightnessBubble {
  public:
+  // Returns the singleton brightness bubble instance.
   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

[msw, 2011/10/21 02:42:06]

remove period after update

[alicet1, 2011/10/21 18:11:36]

Done.

+  // comments at setting_level_bubble.h.
+  void UpdateWithoutShowingBubble(int level, bool enabled);

[msw, 2011/10/21 02:42:06]

SettingLevelBubble::UpdateWithoutShowingBubble takes double |percent| as its
first argument, can you retype and rename this function to be consistent?

[alicet1, 2011/10/21 18:11:36]

Done.

+
+  // 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);
 };