Support "always allow" for runtime script execution

extensions/common/url_pattern_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 "base/memory/scoped_ptr.h"
 #include "extensions/common/url_pattern.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 namespace {
@@ skipping 792 matching lines @@
   EXPECT_TRUE(StrictlyContains(pattern10, pattern11));
   EXPECT_TRUE(StrictlyContains(pattern10, pattern12));
   EXPECT_TRUE(StrictlyContains(pattern10, pattern13));
 
   // More...
   EXPECT_TRUE(StrictlyContains(pattern12, pattern11));
   EXPECT_TRUE(NeitherContains(pattern11, pattern13));
   EXPECT_TRUE(StrictlyContains(pattern12, pattern13));
 }
 
+TEST(ExtensionURLPatternTest, MatchesSingleOrigin) {
+  URLPattern pattern1(URLPattern::SCHEME_ALL);
+  URLPattern pattern2(URLPattern::SCHEME_ALL);
+  URLPattern pattern3(URLPattern::SCHEME_ALL);
+
+  EXPECT_EQ(URLPattern::PARSE_SUCCESS, pattern1.Parse("http://*/*"));
+  EXPECT_EQ(URLPattern::PARSE_SUCCESS, pattern2.Parse("http://*.google.com/*"));
+  EXPECT_EQ(URLPattern::PARSE_SUCCESS, pattern3.Parse("http://www.google.com/*"));
+
+  EXPECT_FALSE(pattern1.MatchesSingleOrigin());
+  EXPECT_FALSE(pattern2.MatchesSingleOrigin());
+  EXPECT_TRUE(pattern3.MatchesSingleOrigin());

[not at google - send to devlin, 2014/08/11 22:59:30]

I think it's unnecessarily complex to be making these in 3 stages. you can do it
all at once, since parsing should succeed:

EXPECT_FALSE(URLPattern(URLPattern::SCHEME_ALL, "http://*/*")
             .MatchesSingleOrigin());
EXPECT_FALSE(URLPattern(URLPattern::SCHEME_ALL, "http://*.google.com/*")
             .MatchesSingleOrigin());
...

also test wildcard scheme like *://google.com, wildcards like *.com, longer
paths, and paths that aren't * - assuming that last one parses, but it would
appear so:

https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/ext...

[gpdavis, 2014/08/12 00:52:06]

Shouldn't MatchesSingleOrigin also check the scheme (and port)?  Because an
origin is a scheme, host, and port, correct?

[not at google - send to devlin, 2014/08/12 19:49:26]

Yes it should check the scheme and port.

+}
+
 }  // namespace