Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1113)

Unified Diff: chrome/browser/extensions/extension_management_browsertest.cc

Issue 8789018: Revert 113047 - Make ExtensionService use ExtensionSet. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/extension_management_browsertest.cc
===================================================================
--- chrome/browser/extensions/extension_management_browsertest.cc (revision 113069)
+++ chrome/browser/extensions/extension_management_browsertest.cc (working copy)
@@ -314,7 +314,7 @@
ExtensionTestMessageListener listener1("v1 installed", false);
ExtensionService* service = browser()->profile()->GetExtensionService();
const size_t size_before = service->extensions()->size();
- ASSERT_TRUE(service->disabled_extensions()->is_empty());
+ ASSERT_TRUE(service->disabled_extensions()->empty());
const Extension* extension =
InstallExtension(basedir.AppendASCII("v1.crx"), 1);
ASSERT_TRUE(extension);
@@ -382,7 +382,7 @@
basedir.AppendASCII("v2.crx"));
const size_t size_before = service->extensions()->size();
- ASSERT_TRUE(service->disabled_extensions()->is_empty());
+ ASSERT_TRUE(service->disabled_extensions()->empty());
PendingExtensionManager* pending_extension_manager =
service->pending_extension_manager();
@@ -465,7 +465,7 @@
basedir.AppendASCII("v2.crx"));
const size_t size_before = service->extensions()->size();
- ASSERT_TRUE(service->disabled_extensions()->is_empty());
+ ASSERT_TRUE(service->disabled_extensions()->empty());
PrefService* prefs = browser()->profile()->GetPrefs();
const ListValue* forcelist =
@@ -520,7 +520,7 @@
service->updater()->set_blacklist_checks_enabled(false);
const size_t size_before = service->extensions()->size();
FilePath basedir = test_data_dir_.AppendASCII("autoupdate");
- ASSERT_TRUE(service->disabled_extensions()->is_empty());
+ ASSERT_TRUE(service->disabled_extensions()->empty());
// Note: This interceptor gets requests on the IO thread.
scoped_refptr<AutoUpdateInterceptor> interceptor(new AutoUpdateInterceptor());
@@ -581,7 +581,7 @@
ASSERT_TRUE(extension);
EXPECT_EQ(Extension::INTERNAL, extension->location());
EXPECT_TRUE(service->IsExtensionEnabled(kExtensionId));
- EXPECT_TRUE(service->disabled_extensions()->is_empty());
+ EXPECT_TRUE(service->disabled_extensions()->empty());
service->DisableExtension(kExtensionId);
EXPECT_EQ(1u, service->disabled_extensions()->size());
@@ -604,5 +604,5 @@
ASSERT_TRUE(extension);
EXPECT_EQ(Extension::EXTERNAL_POLICY_DOWNLOAD, extension->location());
EXPECT_TRUE(service->IsExtensionEnabled(kExtensionId));
- EXPECT_TRUE(service->disabled_extensions()->is_empty());
+ EXPECT_TRUE(service->disabled_extensions()->empty());
}
Property changes on: chrome/browser/extensions/extension_management_browsertest.cc
___________________________________________________________________
Added: svn:mergeinfo
« no previous file with comments | « chrome/browser/extensions/extension_management_api.cc ('k') | chrome/browser/extensions/extension_override_apitest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698