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

Unified Diff: url/gurl.h

Issue 2403713002: Add suborigin logic to url::Origin (Closed)
Patch Set: Minor simplification Created 4 years, 2 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: url/gurl.h
diff --git a/url/gurl.h b/url/gurl.h
index c9111d3684992d457486ed04cb47c616396cf345..032cbd495aeefdecd8c4157413f0aa61007a438a 100644
--- a/url/gurl.h
+++ b/url/gurl.h
@@ -240,7 +240,8 @@ class URL_EXPORT GURL {
// higher-level and more complete semantics. See that function's documentation
// for more detail.
bool SchemeIsCryptographic() const {
- return SchemeIs(url::kHttpsScheme) || SchemeIs(url::kWssScheme);
+ return SchemeIs(url::kHttpsScheme) || SchemeIs(url::kWssScheme) ||
+ SchemeIs(url::kHttpsSuboriginScheme);
Mike West 2016/10/11 07:40:41 This needs tests.
jww 2016/10/11 18:16:27 Done.
}
// Returns true if the scheme is "blob".
@@ -248,6 +249,12 @@ class URL_EXPORT GURL {
return SchemeIs(url::kBlobScheme);
}
+ // Returns true if the scheme indicates a suborigin URL.
+ bool SchemeIsSuborigin() const {
Mike West 2016/10/11 07:40:41 Ditto.
jww 2016/10/11 18:16:27 Done.
+ return SchemeIs(url::kHttpSuboriginScheme) ||
+ SchemeIs(url::kHttpsSuboriginScheme);
+ }
+
// The "content" of the URL is everything after the scheme (skipping the
// scheme delimiting colon). It is an error to get the content of an invalid
// URL: the result will be an empty string.

Powered by Google App Engine
This is Rietveld 408576698