Fix context menus for touchui=1

Fix context menus for touchui=1

Nested tasks are generally allowed for context menus (e.g. in
MenuController::Run, NativeMenuDOMUI::RunMenuAt). Do the same here. This fixes a
X-freeze caused by context menus when touchui=1 with the X message-pump.

BUG=none
TEST=manual, context menus in web-pages should work and not freeze X.

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

[oshima, 2010-11-18 06:59:13]

views menu and domui menu allows nested task for reason.
one qq before lgtm. I thought touch is not using gtk. Am I wrong?

[sadrul, 2010-11-18 13:30:52]

On 2010/11/18 06:59:13, oshima wrote:
> views menu and domui menu allows nested task for reason.
> one qq before lgtm. I thought touch is not using gtk. Am I wrong?

You are correct. GTK+ is being replaced for touchui. While this transition is
happening, there are still some parts that use GTK+, context menus being one of
them. Until it gets replaced, the new message-pump for touchui is somewhat
unusable because context menus cause the entire X to freeze. So this is an
attempt to make it usable during this transition phase. :)

[oshima, 2010-11-18 15:05:21]

LGTM

On Thu, Nov 18, 2010 at 5:30 AM, <sadrul@chromium.org> wrote:

> On 2010/11/18 06:59:13, oshima wrote:
>
>> views menu and domui menu allows nested task for reason.
>> one qq before lgtm. I thought touch is not using gtk. Am I wrong?
>>
>
> You are correct. GTK+ is being replaced for touchui. While this transition
> is
> happening, there are still some parts that use GTK+, context menus being
> one of
> them. Until it gets replaced, the new message-pump for touchui is somewhat
> unusable because context menus cause the entire X to freeze. So this is an
> attempt to make it usable during this transition phase. :)


Got it. Thank you for clarification.

- oshima

>
>
> http://codereview.chromium.org/5206002/
>