Support "always allow" for runtime script execution

extensions/common/url_pattern.cc

Index: extensions/common/url_pattern.cc
diff --git a/extensions/common/url_pattern.cc b/extensions/common/url_pattern.cc
index 735e71e9fa0298704005aa12e3f11fd8fabbc86e..19bfbfecb566702ffc8598deb3d89bde04b83282 100644
--- a/extensions/common/url_pattern.cc
+++ b/extensions/common/url_pattern.cc
@@ -466,6 +466,10 @@ bool URLPattern::ImpliesAllHosts() const {
   return registry_length > 0;
 }
 
+bool URLPattern::MatchesSingleOrigin() const {
+  return scheme_ != "*" && !ImpliesAllHosts() && !match_subdomains_;

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

you should check the port here as well.

and as a nit, it would be nice to arrange the checks in the order that they
appear, so like

return !ImpliesAllHosts() && scheme_ != "*" && port_ != "*" &&
!match_subdomains_;

[gpdavis, 2014/08/12 21:19:55]

When I test this out (logging the visible url's port in the
ActiveScriptController before its origin is added) and hit "always run" on the
google homepage, the port is blank, so it gets added as "*" when it's parsed in
AddOrigin.

How should the port be dealt with?

[not at google - send to devlin, 2014/08/12 23:13:18]

Ah ok. Yeah, forget about port, whatever. Just comment that it's not
interesting.

[gpdavis, 2014/08/13 00:08:24]

Done.

+}
+
 bool URLPattern::MatchesPath(const std::string& test) const {
   // Make the behaviour of OverlapsWith consistent with MatchesURL, which is
   // need to match hosted apps on e.g. 'google.com' also run on 'google.com/'.