Sound volume and bightness bubbles doesn't grab focus anymore.

chrome/browser/chromeos/setting_level_bubble.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/setting_level_bubble.h"
 
 #include <gdk/gdk.h>
 
 #include "base/timer.h"
 #include "chrome/browser/chromeos/login/background_view.h"
@@ skipping 118 matching lines @@
   animation_.Show();
   timeout_timer_.Start(base::TimeDelta::FromSeconds(kBubbleShowTimeoutSec),
                        this, &SettingLevelBubble::OnTimeout);
 }
 
 void SettingLevelBubble::HideBubble() {
   if (bubble_)
     bubble_->Close();
 }
 
+bool SettingLevelBubble::IsShown() const {
+  return bubble_;

[glotov, 2011/07/05 11:10:27]

Does bubble_->Close() zero bubble_ variable?
In other words, why, if the bubble is hidden, bubble_ is NULL?

[SeRya, 2011/07/05 12:09:51]

Yes, SettingLevelBubble::BubbleClosing makes it NULL.

+}
+
 void SettingLevelBubble::UpdateWithoutShowingBubble(int percent) {
   percent = LimitPercent(percent);
 
   previous_percent_ =
       animation_.is_animating() ?
       animation_.GetCurrentValue() :
       current_percent_;
   if (previous_percent_ < 0)
     previous_percent_ = percent;
   current_percent_ = percent;
@@ skipping 31 matching lines @@
 void SettingLevelBubble::AnimationProgressed(const ui::Animation* animation) {
   if (view_) {
     view_->Update(
         ui::Tween::ValueBetween(animation->GetCurrentValue(),
                                 previous_percent_,
                                 current_percent_));
   }
 }
 
 }  // namespace chromeos