diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-12 22:56:52 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-07-12 22:56:52 +0000 |
| commit | c999ae2b2daeb6a0e7f7446ddff39b6691590062 (patch) | |
| tree | dce5402314e7f2a630caa4974233c334621c04bc /views/widget/widget.cc | |
| parent | 9a0a36cfbabd036540d86f1f2acc6eeef7823d16 (diff) | |
| download | chromium_src-c999ae2b2daeb6a0e7f7446ddff39b6691590062.zip chromium_src-c999ae2b2daeb6a0e7f7446ddff39b6691590062.tar.gz chromium_src-c999ae2b2daeb6a0e7f7446ddff39b6691590062.tar.bz2 | |
Revert 92252 - Gets mouse capture to work for menus with pure views. As part of this
I moved what was in menu_host_gtk into native_widget_gtk. Gtk supports
two grab types, both mouse and key. We only want key grab when showing
menus.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/7346003
TBR=sky@chromium.org
Review URL: http://codereview.chromium.org/7350008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@92254 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/widget/widget.cc')
| -rw-r--r-- | views/widget/widget.cc | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/views/widget/widget.cc b/views/widget/widget.cc index a38b59c..d309917 100644 --- a/views/widget/widget.cc +++ b/views/widget/widget.cc @@ -832,9 +832,7 @@ bool Widget::OnMouseEvent(const MouseEvent& event) { switch (event.type()) { case ui::ET_MOUSE_PRESSED: last_mouse_event_was_move_ = false; - // Make sure we're still visible before we attempt capture as the mouse - // press processing may have made the window hide (as happens with menus). - if (GetRootView()->OnMousePressed(event) && IsVisible()) { + if (GetRootView()->OnMousePressed(event)) { is_mouse_button_pressed_ = true; if (!native_widget_->HasMouseCapture()) native_widget_->SetMouseCapture(); |