Allow URLPatterns to omit trailing slash.

Also, a regression had been introduced in earlier changes. Added explicit error codes to Parse(), and removed default constructor, to avoid making the same mistake again.

BUG=49969
TEST=n/a

Review URL: http://codereview.chromium.org/3978007

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63836 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 10 insertions, 0 deletions

diff --git a/chrome/common/extensions/extension_manifests_unittest.cc b/chrome/common/extensions/extension_manifests_unittest.cc
index 5956dd1..e800cd9 100644
--- a/chrome/common/extensions/extension_manifests_unittest.cc
+++ b/chrome/common/extensions/extension_manifests_unittest.cc
@@ -361,3 +361,13 @@ TEST_F(ExtensionManifestTest, GetHomepageURL) {
   extension.reset(LoadAndExpectSuccess("homepage_externally_hosted.json"));
   EXPECT_EQ(GURL(), extension->GetHomepageURL());
 }
+
+TEST_F(ExtensionManifestTest, DefaultPathForExtent) {
+  scoped_ptr<Extension> extension(
+      LoadAndExpectSuccess("default_path_for_extent.json"));
+
+  ASSERT_EQ(1u, extension->web_extent().patterns().size());
+  EXPECT_EQ("/*", extension->web_extent().patterns()[0].path());
+  EXPECT_TRUE(extension->web_extent().ContainsURL(
+      GURL("http://www.google.com/monkey")));
+}