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 bad22a1598488493a9ee343a3d912d3825c7d273..c1a18d1584cecc1c96c3765ba9d064b5d7bce904 100644 |
--- a/chrome/browser/extensions/extension_service_unittest.cc |
+++ b/chrome/browser/extensions/extension_service_unittest.cc |
@@ -1023,7 +1023,8 @@ TEST_F(ExtensionServiceTest, LoadAllExtensionsFromDirectorySuccess) { |
expected_patterns.ClearPatterns(); |
AddPattern(&expected_patterns, "http://*.google.com/*"); |
AddPattern(&expected_patterns, "https://*.google.com/*"); |
- EXPECT_EQ(expected_patterns, extension->permission_set()->explicit_hosts()); |
+ EXPECT_EQ(expected_patterns, |
+ extension->GetActivePermissions()->explicit_hosts()); |
EXPECT_EQ(std::string(good1), loaded_[1]->id()); |
EXPECT_EQ(std::string("My extension 2"), loaded_[1]->name()); |
@@ -1364,7 +1365,7 @@ TEST_F(ExtensionServiceTest, GrantedPermissions) { |
// Make sure there aren't any granted permissions before the |
// extension is installed. |
- scoped_ptr<ExtensionPermissionSet> known_perms( |
+ scoped_refptr<ExtensionPermissionSet> known_perms( |
prefs->GetGrantedPermissions(permissions_crx)); |
EXPECT_FALSE(known_perms.get()); |
@@ -1383,7 +1384,7 @@ TEST_F(ExtensionServiceTest, GrantedPermissions) { |
AddPattern(&expected_host_perms, "http://*.google.com.hk/*"); |
AddPattern(&expected_host_perms, "http://www.example.com/*"); |
- known_perms.reset(prefs->GetGrantedPermissions(extension_id)); |
+ known_perms = prefs->GetGrantedPermissions(extension_id); |
EXPECT_TRUE(known_perms.get()); |
EXPECT_FALSE(known_perms->IsEmpty()); |
EXPECT_EQ(expected_api_perms, known_perms->apis()); |
@@ -1412,7 +1413,7 @@ TEST_F(ExtensionServiceTest, GrantedFullAccessPermissions) { |
std::string extension_id = extension->id(); |
ExtensionPrefs* prefs = service_->extension_prefs(); |
- scoped_ptr<ExtensionPermissionSet> permissions( |
+ scoped_refptr<ExtensionPermissionSet> permissions( |
prefs->GetGrantedPermissions(extension_id)); |
EXPECT_FALSE(permissions->IsEmpty()); |
EXPECT_TRUE(permissions->HasEffectiveFullAccess()); |
@@ -1477,7 +1478,7 @@ TEST_F(ExtensionServiceTest, GrantedAPIAndHostPermissions) { |
ASSERT_TRUE(service_->IsExtensionEnabled(extension_id)); |
ASSERT_FALSE(prefs->DidExtensionEscalatePermissions(extension_id)); |
- scoped_ptr<ExtensionPermissionSet> current_perms( |
+ scoped_refptr<ExtensionPermissionSet> current_perms( |
prefs->GetGrantedPermissions(extension_id)); |
ASSERT_TRUE(current_perms.get()); |
ASSERT_FALSE(current_perms->IsEmpty()); |
@@ -1490,7 +1491,7 @@ TEST_F(ExtensionServiceTest, GrantedAPIAndHostPermissions) { |
// updating the browser to a version which recognizes additional host |
// permissions). |
host_permissions.clear(); |
- current_perms.reset(); |
+ current_perms = NULL; |
host_permissions.insert("http://*.google.com/*"); |
host_permissions.insert("https://*.google.com/*"); |
@@ -1519,7 +1520,7 @@ TEST_F(ExtensionServiceTest, GrantedAPIAndHostPermissions) { |
ASSERT_TRUE(service_->IsExtensionEnabled(extension_id)); |
ASSERT_FALSE(prefs->DidExtensionEscalatePermissions(extension_id)); |
- current_perms.reset(prefs->GetGrantedPermissions(extension_id)); |
+ current_perms = prefs->GetGrantedPermissions(extension_id); |
ASSERT_TRUE(current_perms.get()); |
ASSERT_FALSE(current_perms->IsEmpty()); |
ASSERT_FALSE(current_perms->HasEffectiveFullAccess()); |