rAc: Enforce a 3 second grace period for generated card bubble.

ui/views/widget/widget.cc

 // Copyright (c) 2012 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 "ui/views/widget/widget.h"
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 155 matching lines @@
       disable_inactive_rendering_(false),
       widget_closed_(false),
       saved_show_state_(ui::SHOW_STATE_DEFAULT),
       focus_on_creation_(true),
       is_top_level_(false),
       native_widget_initialized_(false),
       native_widget_destroyed_(false),
       is_mouse_button_pressed_(false),
       is_touch_down_(false),
       last_mouse_event_was_move_(false),
+      capture_explicitly_set_(false),
       root_layers_dirty_(false),
       movement_disabled_(false) {
 }
 
 Widget::~Widget() {
   DestroyRootView();
   if (ownership_ == InitParams::WIDGET_OWNS_NATIVE_WIDGET) {
     delete native_widget_;
   } else {
     DCHECK(native_widget_destroyed_)
@@ skipping 724 matching lines @@
 }
 
 void Widget::SetCapture(View* view) {
   if (internal::NativeWidgetPrivate::IsMouseButtonDown())
     is_mouse_button_pressed_ = true;
   if (internal::NativeWidgetPrivate::IsTouchDown())
     is_touch_down_ = true;
   root_view_->SetMouseHandler(view);
   if (!native_widget_->HasCapture())
     native_widget_->SetCapture();
+  capture_explicitly_set_ = true;

[sky, 2013/09/04 14:10:15]

Do you verify all callers of SetCapture are ok with new behavior? I suspect tab
dragging isn't, but I didn't verify that.

[Evan Stade, 2013/09/04 23:00:38]

You are quite right. Do you like this new approach better?

 }
 
 void Widget::ReleaseCapture() {
   if (native_widget_->HasCapture())
     native_widget_->ReleaseCapture();
+  capture_explicitly_set_ = false;
 }
 
 bool Widget::HasCapture() {
   return native_widget_->HasCapture();
 }
 
 void Widget::TooltipTextChanged(View* view) {
   TooltipManager* manager = native_widget_private()->GetTooltipManager();
   if (manager)
     manager->TooltipTextChanged(view);
@@ skipping 373 matching lines @@
   non_client_view_ = NULL;
   root_view_.reset();
   // Input method has to be destroyed before focus manager.
   input_method_.reset();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // Widget, private:
 
 bool Widget::ShouldReleaseCaptureOnMouseReleased() const {
-  return true;
+  return !capture_explicitly_set_;
 }
 
 void Widget::SetInactiveRenderingDisabled(bool value) {
   if (value == disable_inactive_rendering_)
     return;
 
   disable_inactive_rendering_ = value;
   if (non_client_view_)
     non_client_view_->SetInactiveRenderingDisabled(value);
   native_widget_->SetInactiveRenderingDisabled(value);
@@ skipping 98 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 // internal::NativeWidgetPrivate, NativeWidget implementation:
 
 internal::NativeWidgetPrivate* NativeWidgetPrivate::AsNativeWidgetPrivate() {
   return this;
 }
 
 }  // namespace internal
 }  // namespace views