Extend contextMenus API to support browser/page actions

chrome/browser/ui/app_list/app_context_menu.cc

 // Copyright 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/app_list/app_context_menu.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/extensions/context_menu_matcher.h"
@@ skipping 135 matching lines @@
             USE_LAUNCH_TYPE_FULLSCREEN,
             IDS_APP_CONTEXT_MENU_OPEN_MAXIMIZED);
 #endif
       }
       menu_model_->AddSeparator(ui::NORMAL_SEPARATOR);
     }
 
     // Assign unique IDs to commands added by the app itself.
     int index = USE_LAUNCH_TYPE_COMMAND_END;
     extension_menu_items_->AppendExtensionItems(
-        extensions::MenuItem::ExtensionKey(app_id_), base::string16(), &index);
+        extensions::MenuItem::ExtensionKey(app_id_),
+        base::string16(),
+        &index,
+        true); // include_icons

[Yoyo Zhou, 2014/07/15 23:35:00]

ditto

[gpdavis, 2014/07/16 01:11:10]

Done.

 
     // If at least 1 item was added, add another separator after the list.
     if (index > USE_LAUNCH_TYPE_COMMAND_END)
       menu_model_->AddSeparator(ui::NORMAL_SEPARATOR);
 
     if (!is_platform_app_)
       menu_model_->AddItemWithStringId(OPTIONS, IDS_NEW_TAB_APP_OPTIONS);
 
     menu_model_->AddItemWithStringId(UNINSTALL,
                                      is_platform_app_
@@ skipping 112 matching lines @@
     extension_menu_items_->ExecuteCommand(command_id, NULL,
                                           content::ContextMenuParams());
   } else if (command_id == MENU_NEW_WINDOW) {
     controller_->CreateNewWindow(profile_, false);
   } else if (command_id == MENU_NEW_INCOGNITO_WINDOW) {
     controller_->CreateNewWindow(profile_, true);
   }
 }
 
 }  // namespace app_list