| 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) {
|
|
|