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

Unified Diff: extensions/common/url_pattern_set.cc

Issue 396033002: Support "always allow" for runtime script execution (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Refactoring, minor changes Created 6 years, 4 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: extensions/common/url_pattern_set.cc
diff --git a/extensions/common/url_pattern_set.cc b/extensions/common/url_pattern_set.cc
index ee5ea933830f3cc46e9267c1044942f0487901f2..45ac06753474ef9295ce2fb414775863314a592b 100644
--- a/extensions/common/url_pattern_set.cc
+++ b/extensions/common/url_pattern_set.cc
@@ -142,6 +142,14 @@ void URLPatternSet::ClearPatterns() {
patterns_.clear();
}
+bool URLPatternSet::AddOrigin(int valid_schemes, const GURL& origin) {
+ DCHECK_EQ(origin.GetOrigin(), origin);
+ URLPattern origin_pattern(valid_schemes);
not at google - send to devlin 2014/08/12 19:49:27 add that comment about URL parsing potentially fai
gpdavis 2014/08/12 21:19:55 Done.
+ return (origin_pattern.Parse(origin.GetOrigin().spec()) ==
+ URLPattern::PARSE_SUCCESS) &&
+ AddPattern(origin_pattern);
+}
+
bool URLPatternSet::Contains(const URLPatternSet& other) const {
for (URLPatternSet::const_iterator it = other.begin();
it != other.end(); ++it) {

Powered by Google App Engine
This is Rietveld 408576698