commit 271d01c9ed8873a41b44950bb0ab12e38c9c1696
author mek@chromium.org <mek@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Mon Aug 27 23:48:05 2012
committer mek@chromium.org <mek@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> Mon Aug 27 23:48:05 2012
tree 409afe700e3dabbec242d8fd7b827c6af12b4f1c
parent 9ae9c11cf8cba69967be111c6e4005439b32b116

Platform app context menu items should not be grouped into a submenu.

BUG=141898


Review URL: https://chromiumcodereview.appspot.com/10832349

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

chrome/browser/extensions/platform_app_browsertest.cc

diff --git a/chrome/browser/extensions/platform_app_browsertest.cc b/chrome/browser/extensions/platform_app_browsertest.cc
index 2ee706f..4e92a24 100644
--- a/chrome/browser/extensions/platform_app_browsertest.cc
+++ b/chrome/browser/extensions/platform_app_browsertest.cc
@@ -95,7 +95,7 @@
   // launched a window.
   ASSERT_TRUE(launched_listener.WaitUntilSatisfied());
 
-  // The context_menu app has one context menu item. This, along with a
+  // The context_menu app has two context menu items. These, along with a
   // separator and the developer tools, is all that should be in the menu.
   WebContents* web_contents = GetFirstShellWindowWebContents();
   ASSERT_TRUE(web_contents);
@@ -105,6 +105,7 @@
       params);
   menu->Init();
   ASSERT_TRUE(menu->HasCommandWithId(IDC_EXTENSIONS_CONTEXT_CUSTOM_FIRST));
+  ASSERT_TRUE(menu->HasCommandWithId(IDC_EXTENSIONS_CONTEXT_CUSTOM_FIRST + 1));
   ASSERT_TRUE(menu->HasCommandWithId(IDC_CONTENT_CONTEXT_INSPECTELEMENT));
   ASSERT_TRUE(menu->HasCommandWithId(IDC_RELOAD));
   ASSERT_FALSE(menu->HasCommandWithId(IDC_BACK));