Fix menu crash

Fix menu crash
 Made GetActivatableWindow to follow transient_parent if exists.

BUG=103820
TEST=tbd

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

[oshima, 2011-11-10 23:47:44]

I'm not 100% certain if this is the right fix, but does fix the crash. Tootip
shouldn't be activatable, so maybe we should have separate container for
tooltips?

[oshima, 2011-11-11 00:32:06]

On 2011/11/10 23:47:44, oshima wrote:
> I'm not 100% certain if this is the right fix, but does fix the crash. Tootip
> shouldn't be activatable, so maybe we should have separate container for
> tooltips?

Actually, menu shouldn't be activatable, so real issue may be that we're
activating menu even if we shouldn't. I'm investigating again.

[oshima, 2011-11-11 01:35:51]

Uploaded new patch. PTAL.

I updated GetActivatableWindow to follow transient_paernt because if the window
is menu, the active(atable) window is transient parent but not its parent. Popup
window should work fine as it will haveDefaultContainer or AlwaysOnTopContainer
as a parent.

[Ben Goodger (Google), 2011-11-11 17:14:25]

lgtm

[oshima, 2011-11-11 17:26:16]

since this is crasher, I'll check this in first and add test in follow up
cl.

On Fri, Nov 11, 2011 at 9:14 AM, <ben@chromium.org> wrote:

> lgtm
>
>
>
>
http://codereview.chromium.**org/8523020/<http://codereview.chromium.org/8523...
>