Comment #3 on issue 9843 by [email protected]: Go/Stop Button can switch state while cursor is hovering over it http://code.google.com/p/chromium/issues/detail?id=9843
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=13761
------------------------------------------------------------------------
r13761 | [email protected] | 2009-04-15 11:08:49 -0700 (Wed, 15 Apr
2009) | 6 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.cc?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser.h?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/browser_window.h?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/browser_window_cocoa.h?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/cocoa/browser_window_cocoa.mm?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_window_gtk.cc?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/gtk/browser_window_gtk.h?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/browser_view.cc?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/frame/browser_view.h?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/go_button.cc?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/views/go_button.h?r1=13761&r2=13760
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/test/test_browser_window.h?r1=13761&r2=13760
Fix regression I introduced where Stop/Go button would toggle state
instantly instead of having protections against accidental user actions
while the mouse was hovering the button.
I elected to condense ChangeMode() and ScheduleChangeMode() into one
function, which as a result became pretty simple.
BUG=9843
Review URL: http://codereview.chromium.org/67156
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
