Don't send ViewMsg_ContextMenuClosed to the renderer process for submenus.

Don't send ViewMsg_ContextMenuClosed to the renderer process for submenus.

Note: currently SimpleMenuModel::Delegate doesn't receive the same notifications on all platforms. On Windows and CrOS, MenuWillShow and MenuClosed are sent for all menus (including submenus); while these notifications are not sent for submenus on Linux (haven't tested Mac). This CL makes sure that on Windows and CrOS no ViewMsg_ContextMenuClosed messages are generated for submenus, but it doesn't unify the behavior of SimpleMenuModel::Delegate. I have filed a bug (http://code.google.com/p/chromium/issues/detail?id=90732) to track that issue.

BUG=None
TEST=
- make sure you are running Pepper Flash on either CrOS or Windows;
- go to www.hulu.com;
- open context menu on the flash content;
- move the cursor onto the Quality item to open a submenu;
- move the cursor away;
- the renderer process shouldn't crash.


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

[yzshen1, 2011-07-27 20:51:37]

Hi, Antoine.

Would you please review this change? Thanks!

[piman, 2011-07-27 20:56:02]

LGTM on the pepper side, but adding sky for the UI side, and brettw for
tab_contents.

[brettw, 2011-07-27 21:16:29]

tab contents LGTM

[sky, 2011-07-27 23:46:42]

LGTM

[commit-bot: I haz the power, 2011-07-28 02:15:11]

Change committed as 94416