Modified the toolbar button handlers so that the handlers can revert the locati...

Modified the toolbar button handlers so that the handlers can revert the location bar when the button pressed is RELOAD, FORWARD, or BACK.

Note that this change does not handle F5 nor Ctrl-r key presses. That's intentional. Please see the comment #7 by pkasting in http://crbug.com/15464 .

Windows change: http://codereview.chromium.org/155908
Mac change: http://codereview.chromium.org/164145

BUG=18710
TEST=The same as http://codereview.chromium.org/155908


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=23016

[Yusuke Sato, 2009-08-08 01:43:22]

Evan, can you review this?

[Evan Stade, 2009-08-10 18:04:38]

I think it would be cleaner if you didn't change the BackForwardButtonGtk class
and instead connected to the click event in the toolbar and did the revert
there.

[Yusuke Sato, 2009-08-11 02:26:46]

On 2009/08/10 18:04:38, Evan Stade wrote:
> I think it would be cleaner if you didn't change the BackForwardButtonGtk
class
> and instead connected to the click event in the toolbar and did the revert
> there.

Thanks for the review. Reverted BackForwardButtonGtk changes following your
comment.
Please take another look.

[Evan Stade, 2009-08-11 03:51:56]

lgtm, thanks