| 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 31b77bcb2a923b216dab0ae09c3380bd69e369bd..ac634bd11f4832ae5610bf5d9ca3a20db50f414a 100644
|
| --- a/chrome/browser/extensions/extension_context_menu_model_unittest.cc
|
| +++ b/chrome/browser/extensions/extension_context_menu_model_unittest.cc
|
| @@ -158,7 +158,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);
|
| @@ -166,7 +166,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);
|
|
|