diff options
| author | idanan@chromium.org <idanan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-10 14:05:45 +0000 |
|---|---|---|
| committer | idanan@chromium.org <idanan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-10 14:05:45 +0000 |
| commit | 6cb2b8005e8dbaca2273913234dda828ad51018a (patch) | |
| tree | 8a39c051bc7dac8d5de9f0eb0bf9941ddbe87bef /chrome/browser | |
| parent | f74689457c2b3cb82ba0229ebe55580bafa49a77 (diff) | |
| download | chromium_src-6cb2b8005e8dbaca2273913234dda828ad51018a.zip chromium_src-6cb2b8005e8dbaca2273913234dda828ad51018a.tar.gz chromium_src-6cb2b8005e8dbaca2273913234dda828ad51018a.tar.bz2 | |
Issue 8838: Close box not highlighted
We optimize mouse-mouve event handling by ignoring two mouse-move events in a row
which works in most cases except that when we animate tab-closing on windows we
send a duplicate mouse-move event on purpose to highlight the button. Unfortunately
our optimization kicks in.
Fixed by reseting the flag to optimize this before sending the extra mouse-move.
BUG=8838
Review URL: http://codereview.chromium.org/118454
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@18052 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/views/tabs/tab_strip.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/chrome/browser/views/tabs/tab_strip.cc b/chrome/browser/views/tabs/tab_strip.cc index 2db2c86..d669be3 100644 --- a/chrome/browser/views/tabs/tab_strip.cc +++ b/chrome/browser/views/tabs/tab_strip.cc @@ -346,6 +346,7 @@ class TabStrip::RemoveTabAnimation : public TabStrip::TabAnimation { pt.x -= wr.left; pt.y -= wr.top; + static_cast<views::WidgetWin*>(widget)->ResetLastMouseMoveFlag(); // Return to message loop - otherwise we may disrupt some operation that's // in progress. PostMessage(widget->GetNativeView(), WM_MOUSEMOVE, 0, |