Support "always allow" for runtime script execution

extensions/common/url_pattern_set_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 "extensions/common/url_pattern_set.h"
 
 #include <sstream>
 
 #include "base/values.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 403 matching lines @@
     AddPattern(&expected, yahoo_a);
     AddPattern(&expected, yahoo_b);
     AddPattern(&expected, yahoo_c);
     AddPattern(&expected, reddit_a);
     AddPattern(&expected, reddit_b);
     AddPattern(&expected, reddit_c);
     EXPECT_EQ(expected, result);
   }
 }
 
+TEST(URLPatternSetTest, AddOrigin) {
+  URLPatternSet set;
+  set.AddOrigin(URLPattern::SCHEME_ALL, GURL("https://www.google.com/"));

[not at google - send to devlin, 2014/08/14 23:04:48]

Huh, I would have thought ToOrigin of google.com/ would be google.com, but it's
not failing the DCHECK in there. Ah well.

+  EXPECT_TRUE(set.MatchesURL(GURL("https://www.google.com/foo/bar")));
+  EXPECT_FALSE(set.MatchesURL(GURL("http://www.google.com/foo/bar")));
+  EXPECT_FALSE(set.MatchesURL(GURL("https://en.google.com/foo/bar")));
+  set.ClearPatterns();
+
+  set.AddOrigin(URLPattern::SCHEME_ALL, GURL("http://en.foo.com/"));

[not at google - send to devlin, 2014/08/14 23:04:48]

there's no difference between this and the google.com - they both have exactly
the same structure - so it's not really an interesting check.

what *would* be interesting is testing a URL without a subdomain at least.

[gpdavis, 2014/08/14 23:31:33]

Sure, I can do that.

+  EXPECT_TRUE(set.MatchesURL(GURL("http://en.foo.com/bar")));
+  EXPECT_FALSE(set.MatchesURL(GURL("https://www.foo.com/")));
+  EXPECT_FALSE(set.MatchesURL(GURL("http://www.foo.com/")));
+}
+
 }  // namespace extensions