Convert app_bindings.js to the schema_generated_bindings.js infrastructure.

chrome/common/extensions/api/extension_api_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/common/extensions/api/extension_api.h"
 
 #include <string>
 
 #include "base/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/values.h"
 #include "chrome/common/extensions/extension.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
+namespace {
+
 using extensions::ExtensionAPI;
 
+bool GetSchemasForURLContains(
+    const std::string& api_name, const std::string& url) {
+  ExtensionAPI::SchemaMap schemas;
+  ExtensionAPI::GetInstance()->GetSchemasForURL(GURL(url), &schemas);
+  return schemas.find(api_name) != schemas.end();
+}
+
+bool MatchesURL(
+    const std::string& api_name, const std::string& url) {
+  return ExtensionAPI::GetInstance()->MatchesURL(api_name, GURL(url));
+}
+
 TEST(ExtensionAPI, IsPrivileged) {
   ExtensionAPI* extension_api = ExtensionAPI::GetInstance();
   EXPECT_FALSE(extension_api->IsPrivileged("extension.connect"));
   EXPECT_FALSE(extension_api->IsPrivileged("extension.onConnect"));
 
   // Properties are not supported yet.
   EXPECT_TRUE(extension_api->IsPrivileged("extension.lastError"));
 
   // Default unknown names to privileged for paranoia's sake.
   EXPECT_TRUE(extension_api->IsPrivileged(""));
   EXPECT_TRUE(extension_api->IsPrivileged("<unknown-namespace>"));
   EXPECT_TRUE(extension_api->IsPrivileged("extension.<unknown-member>"));
 
   // Exists, but privileged.
   EXPECT_TRUE(extension_api->IsPrivileged("extension.getViews"));
   EXPECT_TRUE(extension_api->IsPrivileged("history.search"));
 
   // Whole APIs that are unprivileged.
+  EXPECT_FALSE(extension_api->IsPrivileged("app.getDetails"));
+  EXPECT_FALSE(extension_api->IsPrivileged("app.isInstalled"));
   EXPECT_FALSE(extension_api->IsPrivileged("storage.local"));
   EXPECT_FALSE(extension_api->IsPrivileged("storage.local.onChanged"));
   EXPECT_FALSE(extension_api->IsPrivileged("storage.local.set"));
   EXPECT_FALSE(extension_api->IsPrivileged("storage.local.MAX_ITEMS"));
   EXPECT_FALSE(extension_api->IsPrivileged("storage.set"));
 }
 
 TEST(ExtensionAPI, IsWholeAPIPrivileged) {
   ExtensionAPI* extension_api = ExtensionAPI::GetInstance();
 
@@ skipping 28 matching lines @@
   scoped_refptr<Extension> extension(Extension::Create(
       FilePath(), Extension::LOAD, manifest, Extension::NO_FLAGS, &error));
   CHECK(extension.get());
   CHECK(error.empty());
 
   ExtensionAPI::SchemaMap schemas;
   ExtensionAPI::GetInstance()->GetSchemasForExtension(
       *extension, ExtensionAPI::ALL, &schemas);
   EXPECT_EQ(1u, schemas.count("tts"));
 }
+
+TEST(ExtensionAPI, GetSchemasForURL) {
+  // "app" API is available to all URLs that content scripts can be injected.
+  EXPECT_TRUE(GetSchemasForURLContains("app",
+                                       "http://example.com/example.html"));
+  EXPECT_TRUE(GetSchemasForURLContains("app", "https://blah.net"));
+  EXPECT_TRUE(GetSchemasForURLContains("app", "file://somefile.html"));
+
+  // But not internal URLs (for chrome-extension:// the app API is injected by
+  // GetSchemasForExtension).
+  EXPECT_FALSE(GetSchemasForURLContains("app", "about:flags"));
+  EXPECT_FALSE(GetSchemasForURLContains("app", "chrome://flags"));
+  EXPECT_FALSE(GetSchemasForURLContains("app",
+                                        "chrome-extension://fakeextension"));
+
+  // "storage" API (for example) isn't available to any URLs.
+  EXPECT_FALSE(GetSchemasForURLContains("storage",
+                                        "http://example.com/example.html"));
+  EXPECT_FALSE(GetSchemasForURLContains("storage", "https://blah.net"));
+  EXPECT_FALSE(GetSchemasForURLContains("storage", "file://somefile.html"));
+  EXPECT_FALSE(GetSchemasForURLContains("storage", "about:flags"));
+  EXPECT_FALSE(GetSchemasForURLContains("storage", "chrome://flags"));
+  EXPECT_FALSE(GetSchemasForURLContains("storage",
+                                        "chrome-extension://fakeextension"));

[koz (OOO until 15th September), 2012/02/27 03:04:11]

nit: This seems redundant to have the same data passed in to two different
tests. Perhaps consider unifying GetSchemasForURLContains() and MatchesURL()?

[not at google - send to devlin, 2012/02/27 04:44:24]

Yeah, it's ok, you didn't need to soften that with "nit". I was lazy. Done.

+}
+
+TEST(ExtensionAPI, MatchesURL) {
+  // "app" API is available to all URLs that content scripts can be injected.
+  EXPECT_TRUE(MatchesURL("app", "http://example.com/example.html"));
+  EXPECT_TRUE(MatchesURL("app", "https://blah.net"));
+  EXPECT_TRUE(MatchesURL("app", "file://somefile.html"));
+
+  // But not internal URLs (for chrome-extension:// the app API is injected by
+  // GetSchemasForExtension).
+  EXPECT_FALSE(MatchesURL("app", "about:flags"));
+  EXPECT_FALSE(MatchesURL("app", "chrome://flags"));
+  EXPECT_FALSE(MatchesURL("app", "chrome-extension://fakeextension"));
+
+  // "storage" API (for example) isn't available to any URLs.
+  EXPECT_FALSE(MatchesURL("storage", "http://example.com/example.html"));
+  EXPECT_FALSE(MatchesURL("storage", "https://blah.net"));
+  EXPECT_FALSE(MatchesURL("storage", "file://somefile.html"));
+  EXPECT_FALSE(MatchesURL("storage", "about:flags"));
+  EXPECT_FALSE(MatchesURL("storage", "chrome://flags"));
+  EXPECT_FALSE(MatchesURL("storage", "chrome-extension://fakeextension"));
+}
+
+}  // namespace