Fix regression I introduced where Stop/Go button would toggle state instantly...

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
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=13761

[Ben Goodger (Google), 2009-04-15 01:43:38]

OK

[willchan no longer on Chromium, 2009-04-15 18:23:46]

http://codereview.chromium.org/67156/diff/1/4
File chrome/browser/gtk/browser_window_gtk.cc (right):

http://codereview.chromium.org/67156/diff/1/4#newcode434
Line 434: is_loading ? GoButtonGtk::MODE_STOP : GoButtonGtk::MODE_GO, force);
Did you update GoButtonGtk::ChangeMode()?  I don't see it in this changelist.

[Peter Kasting, 2009-04-15 18:25:34]

On 2009/04/15 18:23:46, willchan wrote:
> Did you update GoButtonGtk::ChangeMode()?  I don't see it in this changelist.

No, because I'm dumb.  Will do.