Index: chrome/common/extensions/api/extension_api_unittest.cc |
diff --git a/chrome/common/extensions/api/extension_api_unittest.cc b/chrome/common/extensions/api/extension_api_unittest.cc |
index 125085cf06a4afd48ba88860fb000ba3d7b2b0c1..8584a1e376c08ef1f439c7d40600fbda6d9a5e7f 100644 |
--- a/chrome/common/extensions/api/extension_api_unittest.cc |
+++ b/chrome/common/extensions/api/extension_api_unittest.cc |
@@ -235,32 +235,29 @@ TEST(ExtensionAPI, ExtensionWithUnprivilegedAPIs) { |
scoped_ptr<ExtensionAPI> extension_api( |
ExtensionAPI::CreateWithDefaultConfiguration()); |
- scoped_ptr<std::set<std::string> > privileged_apis = |
- extension_api->GetAPIsForContext( |
- Feature::BLESSED_EXTENSION_CONTEXT, extension.get(), GURL()); |
+ std::set<std::string> privileged_apis = extension_api->GetAPIsForContext( |
+ Feature::BLESSED_EXTENSION_CONTEXT, extension.get(), GURL()); |
- scoped_ptr<std::set<std::string> > unprivileged_apis = |
- extension_api->GetAPIsForContext( |
- Feature::UNBLESSED_EXTENSION_CONTEXT, extension.get(), GURL()); |
+ std::set<std::string> unprivileged_apis = extension_api->GetAPIsForContext( |
+ Feature::UNBLESSED_EXTENSION_CONTEXT, extension.get(), GURL()); |
- scoped_ptr<std::set<std::string> > content_script_apis = |
- extension_api->GetAPIsForContext( |
- Feature::CONTENT_SCRIPT_CONTEXT, extension.get(), GURL()); |
+ std::set<std::string> content_script_apis = extension_api->GetAPIsForContext( |
+ Feature::CONTENT_SCRIPT_CONTEXT, extension.get(), GURL()); |
// "storage" is completely unprivileged. |
- EXPECT_EQ(1u, privileged_apis->count("storage")); |
- EXPECT_EQ(1u, unprivileged_apis->count("storage")); |
- EXPECT_EQ(1u, content_script_apis->count("storage")); |
+ EXPECT_EQ(1u, privileged_apis.count("storage")); |
+ EXPECT_EQ(1u, unprivileged_apis.count("storage")); |
+ EXPECT_EQ(1u, content_script_apis.count("storage")); |
// "extension" is partially unprivileged. |
- EXPECT_EQ(1u, privileged_apis->count("extension")); |
- EXPECT_EQ(1u, unprivileged_apis->count("extension")); |
- EXPECT_EQ(1u, content_script_apis->count("extension")); |
+ EXPECT_EQ(1u, privileged_apis.count("extension")); |
+ EXPECT_EQ(1u, unprivileged_apis.count("extension")); |
+ EXPECT_EQ(1u, content_script_apis.count("extension")); |
// "history" is entirely privileged. |
- EXPECT_EQ(1u, privileged_apis->count("history")); |
- EXPECT_EQ(0u, unprivileged_apis->count("history")); |
- EXPECT_EQ(0u, content_script_apis->count("history")); |
+ EXPECT_EQ(1u, privileged_apis.count("history")); |
+ EXPECT_EQ(0u, unprivileged_apis.count("history")); |
+ EXPECT_EQ(0u, content_script_apis.count("history")); |
} |
TEST(ExtensionAPI, ExtensionWithDependencies) { |
@@ -271,10 +268,10 @@ TEST(ExtensionAPI, ExtensionWithDependencies) { |
CreateExtensionWithPermission("ttsEngine"); |
scoped_ptr<ExtensionAPI> api( |
ExtensionAPI::CreateWithDefaultConfiguration()); |
- scoped_ptr<std::set<std::string> > apis = api->GetAPIsForContext( |
+ std::set<std::string> apis = api->GetAPIsForContext( |
Feature::BLESSED_EXTENSION_CONTEXT, extension.get(), GURL()); |
- EXPECT_EQ(1u, apis->count("ttsEngine")); |
- EXPECT_EQ(1u, apis->count("tts")); |
+ EXPECT_EQ(1u, apis.count("ttsEngine")); |
+ EXPECT_EQ(1u, apis.count("tts")); |
} |
// Conversely, extension with the "tts" permission but not the "ttsEngine" |
@@ -284,18 +281,18 @@ TEST(ExtensionAPI, ExtensionWithDependencies) { |
CreateExtensionWithPermission("tts"); |
scoped_ptr<ExtensionAPI> api( |
ExtensionAPI::CreateWithDefaultConfiguration()); |
- scoped_ptr<std::set<std::string> > apis = api->GetAPIsForContext( |
+ std::set<std::string> apis = api->GetAPIsForContext( |
Feature::BLESSED_EXTENSION_CONTEXT, extension.get(), GURL()); |
- EXPECT_EQ(0u, apis->count("ttsEngine")); |
- EXPECT_EQ(1u, apis->count("tts")); |
+ EXPECT_EQ(0u, apis.count("ttsEngine")); |
+ EXPECT_EQ(1u, apis.count("tts")); |
} |
} |
bool MatchesURL( |
ExtensionAPI* api, const std::string& api_name, const std::string& url) { |
- scoped_ptr<std::set<std::string> > apis = |
+ std::set<std::string> apis = |
api->GetAPIsForContext(Feature::WEB_PAGE_CONTEXT, NULL, GURL(url)); |
- return apis->count(api_name); |
+ return apis.count(api_name); |
} |
TEST(ExtensionAPI, URLMatching) { |