Re-enable storage for Suborigins.

content/common/origin_util.cc

 // Copyright 2015 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 "content/public/common/origin_util.h"
 
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "base/stl_util.h"
 #include "content/public/common/content_client.h"
@@ skipping 72 matching lines @@
     return true;
   }
 
   return false;
 }
 
 void ResetSchemesAndOriginsWhitelistForTesting() {
   g_trustworthy_whitelist.Get().Reset();
 }
 
+bool HasSuborigin(const GURL& url) {
+  return url.scheme() == "http-so" || url.scheme() == "https-so";

[nasko, 2016/09/26 22:54:54]

Shouldn't the GURL be valid before we start poking it?

Please use the defined constants instead of hardcoded strings.

[jww, 2016/09/28 23:35:14]

Done.

+}
+
+std::string SuboriginFromUrl(const GURL& url) {
+  if (!HasSuborigin(url))
+    return "";
+  size_t suborigin_end = url.host().find(".");
+  return url.host().substr(0, suborigin_end);
+}
+
+std::string HostFromUrlStripSuborigin(const GURL& url) {

[nasko, 2016/09/26 22:54:54]

Unused method?

[jww, 2016/09/28 23:35:14]

Done.

+  std::string host = url.host();
+  size_t suborigin_end = host.find(".");
+  if (suborigin_end == std::string::npos)
+    return host;
+  return url.host().substr(suborigin_end + 1,
+                           host.length() - suborigin_end - 1);
+}
+
+std::string SchemeFromUrlStripSuborigin(const GURL& url) {

[nasko, 2016/09/26 22:54:55]

Unused method?

[jww, 2016/09/28 23:35:15]

Done.

+  if (!HasSuborigin(url))
+    return "";
+
+  if (url.scheme() == "http-so")
+    return "http";
+
+  if (url.scheme() == "https-so")
+    return "https";
+
+  NOTREACHED();
+  return "";
+}
+
+GURL UrlStripSuborigin(const GURL& url) {
+  if (!HasSuborigin(url))
+    return url;
+
+  GURL::Replacements replacements;
+  if (url.scheme() == "http-so") {
+    replacements.SetSchemeStr("http");

[nasko, 2016/09/26 22:54:54]

Use constants instead of hardcoded strings.

[jww, 2016/09/28 23:35:14]

Done.

+  } else {
+    DCHECK(url.scheme() == "https-so");
+    replacements.SetSchemeStr("https");
+  }
+
+  size_t suborigin_end = url.host().find(".");
+  std::string new_host(url.host().substr(
+      suborigin_end + 1, url.host().length() - suborigin_end - 1));
+  replacements.SetHostStr(new_host);
+
+  return url.ReplaceComponents(replacements);
+}
+
 }  // namespace content