From 6cb2b8005e8dbaca2273913234dda828ad51018a Mon Sep 17 00:00:00 2001 From: "idanan@chromium.org" Date: Wed, 10 Jun 2009 14:05:45 +0000 Subject: 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 --- views/widget/widget_win.h | 8 ++++++++ 1 file changed, 8 insertions(+) (limited to 'views') 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. -- cgit v1.1