Add frameId to contextMenus.onClicked / onclick.

chrome/browser/ui/app_list/extension_app_context_menu.cc

Index: chrome/browser/ui/app_list/extension_app_context_menu.cc
diff --git a/chrome/browser/ui/app_list/extension_app_context_menu.cc b/chrome/browser/ui/app_list/extension_app_context_menu.cc
index 65a3a483617f78af2cf62e3f0bc9884d56f20302..39d939b680d3845d08d90965eb8b9dfeea2c2ea1 100644
--- a/chrome/browser/ui/app_list/extension_app_context_menu.cc
+++ b/chrome/browser/ui/app_list/extension_app_context_menu.cc
@@ -237,7 +237,7 @@ void ExtensionAppContextMenu::ExecuteCommand(int command_id, int event_flags) {
     controller()->UninstallApp(profile(), app_id());
   } else if (extensions::ContextMenuMatcher::IsExtensionsCustomCommandId(
                  command_id)) {
-    extension_menu_items_->ExecuteCommand(command_id, nullptr,
+    extension_menu_items_->ExecuteCommand(command_id, nullptr, nullptr,

[sky, 2016/03/09 22:44:40]

Don't extensions have an associated RFH?

[robwu, 2016/03/09 22:52:20]

It seems that this context menu item is context-less (at the next line,
ContextMenuParams()'s default constructor is used). This context menu is shown
at chrome://apps, so not attributing the context menu entry to the tab and/or
frame looks OK to me.

                                           content::ContextMenuParams());
   } else if (command_id == MENU_NEW_WINDOW) {
     controller()->CreateNewWindow(profile(), false);