Index: chrome/browser/extensions/extension_context_menu_model_unittest.cc |
diff --git a/chrome/browser/extensions/extension_context_menu_model_unittest.cc b/chrome/browser/extensions/extension_context_menu_model_unittest.cc |
index e230364c45e0a4feb70fc604203761dd1712ec80..4cee7a97e19d552fd7f2b0cbee1c84b7358a3b6e 100644 |
--- a/chrome/browser/extensions/extension_context_menu_model_unittest.cc |
+++ b/chrome/browser/extensions/extension_context_menu_model_unittest.cc |
@@ -147,7 +147,7 @@ TEST_F(ExtensionContextMenuModelTest, RequiredInstallationsDisablesItems) { |
system->management_policy()->UnregisterAllProviders(); |
// Uninstallation should be, by default, enabled. |
- ASSERT_TRUE(menu->IsCommandIdEnabled(ExtensionContextMenuModel::UNINSTALL)); |
+ EXPECT_TRUE(menu->IsCommandIdEnabled(ExtensionContextMenuModel::UNINSTALL)); |
TestManagementPolicyProvider policy_provider( |
TestManagementPolicyProvider::PROHIBIT_MODIFY_STATUS); |
@@ -155,7 +155,14 @@ TEST_F(ExtensionContextMenuModelTest, RequiredInstallationsDisablesItems) { |
// If there's a policy provider that requires the extension stay enabled, then |
// uninstallation should be disabled. |
- ASSERT_FALSE(menu->IsCommandIdEnabled(ExtensionContextMenuModel::UNINSTALL)); |
+ EXPECT_FALSE(menu->IsCommandIdEnabled(ExtensionContextMenuModel::UNINSTALL)); |
+ int uninstall_index = |
+ menu->GetIndexOfCommandId(ExtensionContextMenuModel::UNINSTALL); |
+ // There should also be an icon to visually indicate why uninstallation is |
+ // forbidden. |
+ gfx::Image icon; |
+ EXPECT_TRUE(menu->GetIconAt(uninstall_index, &icon)); |
+ EXPECT_FALSE(icon.IsEmpty()); |
// Don't leave |policy_provider| dangling. |
system->management_policy()->UnregisterProvider(&policy_provider); |