Fixing focussing issue when bubble opens

Fixing focussing issue when bubble opens


BUG=141493
TEST=unit test


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

[Mr4D (OOO till 08-26), 2012-08-14 16:17:58]

Fixed activation issue. Please have a look!

[sky, 2012-08-14 16:53:13]

LGTM

[sky, 2012-08-14 16:53:46]

After this patch do mouse clicks on the buttons still work?

[Mr4D (OOO till 08-26), 2012-08-14 18:11:06]

Yes. Granted. I was surprised. But this was the only way to get it to not take
the focus and it seems to work. Is there a better way? Oh - do you want me to
add another unit test for that?

[sky, 2012-08-14 20:07:26]

I just wanted to make sure this didn't cause problems. And yes on the unit
test.

  -Scott

On Tue, Aug 14, 2012 at 11:11 AM,  <skuhne@chromium.org> wrote:
> Yes. Granted. I was surprised. But this was the only way to get it to not
> take
> the focus and it seems to work. Is there a better way? Oh - do you want me
> to
> add another unit test for that?
>
> http://codereview.chromium.org/10825343/

[Mr4D (OOO till 08-26), 2012-08-15 14:34:34]

I just realized - the unit test for this is already in place. (named
"MaximizeLeftByButton").

Will commit now. Thanks!

[commit-bot: I haz the power, 2012-08-15 14:34:50]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/skuhne@chromium.org/10825343/1

[commit-bot: I haz the power, 2012-08-15 16:53:50]

Change committed as 151707