Index: chrome/browser/extensions/extension_service_unittest.cc |
diff --git a/chrome/browser/extensions/extension_service_unittest.cc b/chrome/browser/extensions/extension_service_unittest.cc |
index 58933ef1f4e0c4bdbca6e7224b13f23969064c36..bbbff27cd49548bf7c9fbfc000b7342a6c604ee7 100644 |
--- a/chrome/browser/extensions/extension_service_unittest.cc |
+++ b/chrome/browser/extensions/extension_service_unittest.cc |
@@ -165,7 +165,6 @@ |
using extensions::Manifest; |
using extensions::PermissionSet; |
using extensions::TestExtensionSystem; |
-using extensions::UnloadedExtensionInfo; |
using extensions::URLPatternSet; |
namespace keys = extensions::manifest_keys; |
@@ -667,12 +666,10 @@ |
: public ExtensionServiceTestBase, public content::NotificationObserver { |
public: |
ExtensionServiceTest() |
- : unloaded_reason_(UnloadedExtensionInfo::REASON_UNDEFINED), |
- installed_(NULL), |
+ : installed_(NULL), |
was_update_(false), |
override_external_install_prompt_( |
- FeatureSwitch::prompt_for_external_extensions(), |
- false) { |
+ FeatureSwitch::prompt_for_external_extensions(), false) { |
registrar_.Add(this, |
chrome::NOTIFICATION_EXTENSION_LOADED_DEPRECATED, |
content::NotificationService::AllSources()); |
@@ -697,11 +694,10 @@ |
} |
case chrome::NOTIFICATION_EXTENSION_UNLOADED_DEPRECATED: { |
- UnloadedExtensionInfo* unloaded_info = |
- content::Details<UnloadedExtensionInfo>(details).ptr(); |
- const Extension* e = unloaded_info->extension; |
+ const Extension* e = |
+ content::Details<extensions::UnloadedExtensionInfo>( |
+ details)->extension; |
unloaded_id_ = e->id(); |
- unloaded_reason_ = unloaded_info->reason; |
extensions::ExtensionList::iterator i = |
std::find(loaded_.begin(), loaded_.end(), e); |
// TODO(erikkay) fix so this can be an assert. Right now the tests |
@@ -1254,7 +1250,6 @@ |
protected: |
extensions::ExtensionList loaded_; |
std::string unloaded_id_; |
- UnloadedExtensionInfo::Reason unloaded_reason_; |
const Extension* installed_; |
bool was_update_; |
std::string old_name_; |
@@ -4172,7 +4167,6 @@ |
EXPECT_EQ(1u, registry_->enabled_extensions().size()); |
UninstallExtension(good_crx, false); |
EXPECT_EQ(0u, registry_->enabled_extensions().size()); |
- EXPECT_EQ(UnloadedExtensionInfo::REASON_UNINSTALL, unloaded_reason_); |
} |
TEST_F(ExtensionServiceTest, UninstallTerminatedExtension) { |
@@ -4180,7 +4174,6 @@ |
InstallCRX(data_dir_.AppendASCII("good.crx"), INSTALL_NEW); |
TerminateExtension(good_crx); |
UninstallExtension(good_crx, false); |
- EXPECT_EQ(UnloadedExtensionInfo::REASON_TERMINATE, unloaded_reason_); |
} |
// Tests the uninstaller helper. |
@@ -4188,7 +4181,6 @@ |
InitializeEmptyExtensionService(); |
InstallCRX(data_dir_.AppendASCII("good.crx"), INSTALL_NEW); |
UninstallExtension(good_crx, true); |
- EXPECT_EQ(UnloadedExtensionInfo::REASON_UNINSTALL, unloaded_reason_); |
} |
TEST_F(ExtensionServiceTest, UninstallExtensionHelperTerminated) { |
@@ -4196,7 +4188,6 @@ |
InstallCRX(data_dir_.AppendASCII("good.crx"), INSTALL_NEW); |
TerminateExtension(good_crx); |
UninstallExtension(good_crx, true); |
- EXPECT_EQ(UnloadedExtensionInfo::REASON_TERMINATE, unloaded_reason_); |
} |
// An extension disabled because of unsupported requirements should re-enabled |
@@ -6926,24 +6917,3 @@ |
EXPECT_EQ(expected_disabled_extensions, |
registry_->disabled_extensions().GetIDs()); |
} |
- |
-// Tests a profile being destroyed correctly disables extensions. |
-TEST_F(ExtensionServiceTest, DestroyingProfileClearsExtensions) { |
- InitializeEmptyExtensionService(); |
- |
- InstallCRX(data_dir_.AppendASCII("good.crx"), INSTALL_NEW); |
- EXPECT_NE(UnloadedExtensionInfo::REASON_PROFILE_SHUTDOWN, unloaded_reason_); |
- EXPECT_EQ(1u, registry_->enabled_extensions().size()); |
- EXPECT_EQ(0u, registry_->disabled_extensions().size()); |
- EXPECT_EQ(0u, registry_->terminated_extensions().size()); |
- EXPECT_EQ(0u, registry_->blacklisted_extensions().size()); |
- |
- service_->Observe(chrome::NOTIFICATION_PROFILE_DESTRUCTION_STARTED, |
- content::Source<Profile>(profile_.get()), |
- content::NotificationService::NoDetails()); |
- EXPECT_EQ(UnloadedExtensionInfo::REASON_PROFILE_SHUTDOWN, unloaded_reason_); |
- EXPECT_EQ(0u, registry_->enabled_extensions().size()); |
- EXPECT_EQ(0u, registry_->disabled_extensions().size()); |
- EXPECT_EQ(0u, registry_->terminated_extensions().size()); |
- EXPECT_EQ(0u, registry_->blacklisted_extensions().size()); |
-} |