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 /views/widget | |
| 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 'views/widget')
| -rw-r--r-- | views/widget/widget_win.h | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/views/widget/widget_win.h b/views/widget/widget_win.h index cec2537..2a9562c 100644 --- a/views/widget/widget_win.h +++ b/views/widget/widget_win.h @@ -318,6 +318,14 @@ class WidgetWin : public Widget, return ::GetClientRect(GetNativeView(), rect); } + // Resets the last move flag so that we can go around the optimization + // that disregards duplicate mouse moves when ending animation requires + // a new hit-test to do some highlighting as in TabStrip::RemoveTabAnimation + // to cause the close button to highlight. + void ResetLastMouseMoveFlag() { + last_mouse_event_was_move_ = false; + } + protected: // Call close instead of this to Destroy the window. |