Lazy load chrome.* APIs

chrome/common/extensions/api/extension_api_unittest.cc

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 106656c8f16e1b253e44f7d5040f4fecf4863ccc..21256b084a8f4dcf008d4a68289ece010f595938 100644
--- a/chrome/common/extensions/api/extension_api_unittest.cc
+++ b/chrome/common/extensions/api/extension_api_unittest.cc
@@ -233,32 +233,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) {
@@ -269,10 +266,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"
@@ -282,18 +279,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) {