Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(9975)

Unified Diff: chrome/common/extensions/manifest_tests/extension_manifests_web_accessible_resources_unittest.cc

Issue 11821009: move 'web_accessible_resources' parsing out of Extension class. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@url_parse
Patch Set: Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/common/extensions/manifest_tests/extension_manifests_web_accessible_resources_unittest.cc
diff --git a/chrome/common/extensions/manifest_tests/extension_manifests_web_accessible_resources_unittest.cc b/chrome/common/extensions/manifest_tests/extension_manifests_web_accessible_resources_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..352b2f06e2746d67cfdd1cf0d3a95bd78a37f52a
--- /dev/null
+++ b/chrome/common/extensions/manifest_tests/extension_manifests_web_accessible_resources_unittest.cc
@@ -0,0 +1,90 @@
+// 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/extension_manifest_constants.h"
+#include "chrome/common/extensions/manifest_handler.h"
+#include "chrome/common/extensions/manifest_tests/extension_manifest_test.h"
+#include "chrome/common/extensions/manifest_web_handler.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using extensions::Extension;
+using extensions::WebAccessibleResourcesInfo;
+
+class WebAccesibleResourcesManifestTest : public ExtensionManifestTest {
+ virtual void SetUp() OVERRIDE {
+ ExtensionManifestTest::SetUp();
+ extensions::ManifestHandler::Register(
+ extension_manifest_keys::kWebAccessibleResources,
+ new extensions::WebAccessibleResourcesHandler);
+ }
+};
+
+TEST_F(WebAccesibleResourcesManifestTest, WebAccessibleResources) {
+ // Manifest version 2 with web accessible resources specified.
+ scoped_refptr<Extension> extension1(
+ LoadAndExpectSuccess("web_accessible_resources_1.json"));
+
+ // Manifest version 2 with no web accessible resources.
+ scoped_refptr<Extension> extension2(
+ LoadAndExpectSuccess("web_accessible_resources_2.json"));
+
+ // Default manifest version with web accessible resources specified.
+ scoped_refptr<Extension> extension3(
+ LoadAndExpectSuccess("web_accessible_resources_3.json"));
+
+ // Default manifest version with no web accessible resources.
+ scoped_refptr<Extension> extension4(
+ LoadAndExpectSuccess("web_accessible_resources_4.json"));
+
+ // Default manifest version with wildcard web accessible resource.
+ scoped_refptr<Extension> extension5(
+ LoadAndExpectSuccess("web_accessible_resources_5.json"));
+
+ // Default manifest version with wildcard with specific path and extension.
+ scoped_refptr<Extension> extension6(
+ LoadAndExpectSuccess("web_accessible_resources_6.json"));
+
+ EXPECT_TRUE(
+ WebAccessibleResourcesInfo::HasWebAccessibleResources(extension1));
+ EXPECT_FALSE(
+ WebAccessibleResourcesInfo::HasWebAccessibleResources(extension2));
+ EXPECT_TRUE(
+ WebAccessibleResourcesInfo::HasWebAccessibleResources(extension3));
+ EXPECT_FALSE(
+ WebAccessibleResourcesInfo::HasWebAccessibleResources(extension4));
+ EXPECT_TRUE(
+ WebAccessibleResourcesInfo::HasWebAccessibleResources(extension5));
+ EXPECT_TRUE(
+ WebAccessibleResourcesInfo::HasWebAccessibleResources(extension6));
+
+ EXPECT_TRUE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "test", extension1));
+ EXPECT_FALSE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "none", extension1));
+
+ EXPECT_FALSE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "test", extension2));
+
+ EXPECT_TRUE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "test", extension3));
+ EXPECT_FALSE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "none", extension3));
+
+ EXPECT_TRUE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "test", extension4));
+ EXPECT_TRUE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "none", extension4));
+
+ EXPECT_TRUE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "anything", extension5));
+ EXPECT_TRUE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "path/anything", extension5));
+
+ EXPECT_TRUE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "path/anything.ext", extension6));
+ EXPECT_FALSE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "anything.ext", extension6));
+ EXPECT_FALSE(WebAccessibleResourcesInfo::IsResourceWebAccessible(
+ "path/anything.badext", extension6));
+}

Powered by Google App Engine
This is Rietveld 408576698