Add suborigin logic to url::Origin

url/gurl.h

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);
   }
 
   // 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.

[nasko, 2016/10/13 22:16:45]

nit: An URL can't really be suborigin, just the serialized version of an Origin.
It will be good to make the comment explicit that the GURL is for serialized
Origin not something real on the web.

[jww, 2016/10/14 01:08:57]

I've updated the wording, but you might want to double check that it's clear
enough.

[nasko, 2016/10/14 14:12:41]

Acknowledged.

+  bool SchemeIsSuborigin() const {
+    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.