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

Unified Diff: chrome/browser/extensions/api/declarative/rules_registry_with_cache_unittest.cc

Issue 2738553002: [Extensions] Log instances of invalid extensions being added (Closed)
Patch Set: . Created 3 years, 9 months 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/api/declarative/rules_registry_with_cache_unittest.cc
diff --git a/chrome/browser/extensions/api/declarative/rules_registry_with_cache_unittest.cc b/chrome/browser/extensions/api/declarative/rules_registry_with_cache_unittest.cc
index e8798fd378a5511ea6bc679cf870818f14a37404..1d609a48005e0dd0a9c982ba948c8f0ea20c3544 100644
--- a/chrome/browser/extensions/api/declarative/rules_registry_with_cache_unittest.cc
+++ b/chrome/browser/extensions/api/declarative/rules_registry_with_cache_unittest.cc
@@ -342,13 +342,9 @@ TEST_F(RulesRegistryWithCacheTest, RulesPreservedAcrossRestart) {
// 1. Add an extension, before rules registry gets created.
std::string error;
- scoped_refptr<Extension> extension(
- LoadManifestUnchecked("permissions",
- "web_request_all_host_permissions.json",
- Manifest::INVALID_LOCATION,
- Extension::NO_FLAGS,
- extension1_->id(),
- &error));
+ scoped_refptr<Extension> extension(LoadManifestUnchecked(
+ "permissions", "web_request_all_host_permissions.json",
+ Manifest::UNPACKED, Extension::NO_FLAGS, extension1_->id(), &error));
ASSERT_TRUE(error.empty());
extension_service->AddExtension(extension.get());
EXPECT_TRUE(extensions::ExtensionRegistry::Get(env_.profile())

Powered by Google App Engine
This is Rietveld 408576698