Dismiss the bookmark manager when Ctrl-W is pressed

Dismiss the bookmark manager when Ctrl-W is pressed.

The bookmark manager is a window not a dialog. It should be dismissed by Ctrl-W
not by Esc as well as browser tabs in the main window.

TEST=Open the bookmark manager, then press Ctrl-W. It should be closed.

BUG=4030

[tyoshino-do-not-use, 2009-05-19 08:35:22]

Hi Venkat, (and CC-ing sky@ and ben@)

I'd like you to do code review.
I agree with you about http://code.google.com/p/chromium/issues/detail?id=4030.
The behavior of dialogs should be consistent.

Hi Sky and Ben,

Could you explain why you objected the suggestion on BUG4030? There seems no
justification provided.

Thank you.

[sky, 2009-05-22 23:41:16]

On 2009/05/19 08:35:22, tyoshino wrote:
> Hi Venkat, (and CC-ing sky@ and ben@)
> 
> I'd like you to do code review.
> I agree with you about
http://code.google.com/p/chromium/issues/detail?id=4030.
> The behavior of dialogs should be consistent.
> 
> Hi Sky and Ben,
> 
> Could you explain why you objected the suggestion on BUG4030? There seems no
> justification provided.
> 
> Thank you.

I updated the bug with my rationale. If you have any other questions, just asks.

Sorry about not responding to this right away. I updated the bug after you sent
the review, then forgot to respond here as well.

  -Scott

[tyoshino-do-not-use, 2009-05-27 03:33:52]

Thank you for the update. As discussed on bug 4030, I've updated this CL.

Hi Venkat, as you said outside this thread, I'll follow Scott and Ben for code
review.

Thanks,

[sky, 2009-05-27 03:51:33]

LGTM

[tyoshino-do-not-use, 2009-05-27 03:56:52]

Thank you for review. Could you please check this in on behalf of me? I don't
have commit access.

Thanks,

[sky, 2009-05-27 14:35:04]

Certainly. Committed in r16982.

  -Scott

[sky, 2009-05-27 16:37:59]

*SIGH*
This was backed out in 16985 because the mac build broke. It wasn't this cl's
fault though, so I landed it again in 16987. Lets hope it sticks.

  -Scott

[tyoshino-do-not-use, 2009-05-28 04:16:25]

On 2009/05/27 16:37:59, sky wrote:
> *SIGH*
> This was backed out in 16985 because the mac build broke. It wasn't this cl's
> fault though, so I landed it again in 16987. Lets hope it sticks.
> 
>   -Scott

Looks successful. Thanks.