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.


Review URL: http://codereview.chromium.org/7492054

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@94416 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 2 insertions, 2 deletions

diff --git a/ui/base/models/simple_menu_model.h b/ui/base/models/simple_menu_model.h
index de233fd..9f7136d 100644
--- a/ui/base/models/simple_menu_model.h
+++ b/ui/base/models/simple_menu_model.h
@@ -50,10 +50,10 @@ class UI_API SimpleMenuModel : public MenuModel {
     virtual void ExecuteCommand(int command_id) = 0;
 
     // Notifies the delegate that the menu is about to show.
-    virtual void MenuWillShow();
+    virtual void MenuWillShow(SimpleMenuModel* source);
 
     // Notifies the delegate that the menu has closed.
-    virtual void MenuClosed();
+    virtual void MenuClosed(SimpleMenuModel* source);
 
    protected:
     virtual ~Delegate() {}