Remove the accelerator group when closing the browser window.

Remove the accelerator group when closing the browser window.

This is something that GTK should be doing automatically when the window is
destroyed, but it doesn't seem to happen.  Detach it ourselves so that we
don't leak the closures attached to the acclerator group.

BUG=10737

[Evan Stade, 2009-07-22 18:21:59]

don't we have other accel groups that we use that should get the same treatment?

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

http://codereview.chromium.org/155921/diff/1/2#newcode1276
Line 1276: GtkAccelGroup* accel_group_ = gtk_accel_group_new();
s/GtkAccelGroup* //

[Evan Stade, 2009-07-22 20:50:51]

also, should we remove the suppression

[Dean McNamee, 2009-07-23 13:00:06]

We probably have others, one bug at a time.

Btw, was that a LG?

On 2009/07/22 20:50:51, Evan Stade wrote:
> also, should we remove the suppression

[Evan Stade, 2009-07-23 20:00:08]

On Thu, Jul 23, 2009 at 6:00 AM, <deanm@chromium.org> wrote:
> We probably have others, one bug at a time.
>
> Btw, was that a LG?

if you remove the relevant suppression (if any) LG

>
> On 2009/07/22 20:50:51, Evan Stade wrote:
>>
>> also, should we remove the suppression
>
>
>
> http://codereview.chromium.org/155921
>