Commit 64627047 authored by Yutaka Hirano's avatar Yutaka Hirano Committed by Commit Bot

Remove SchemeRegistry::RegisterURLSchemeAsCORSEnabled

It's not used in chromium, and we need to remove it because we are
moving CORS logic to the network service.

Bug: 800669
Change-Id: I78d4b9ed641d3dca54ad80954872082e2450d2ef
Reviewed-on: https://chromium-review.googlesource.com/1157364
Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: default avatarMike West <mkwst@chromium.org>
Reviewed-by: default avatarTakashi Toyoshima <toyoshim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580838}
parent d0a3dc6f
......@@ -224,11 +224,6 @@ bool SchemeRegistry::ShouldTreatURLSchemeAsNotAllowingJavascriptURLs(
scheme);
}
void SchemeRegistry::RegisterURLSchemeAsCORSEnabled(const String& scheme) {
DCHECK_EQ(scheme, scheme.LowerASCII());
GetMutableURLSchemesRegistry().cors_enabled_schemes.insert(scheme);
}
bool SchemeRegistry::ShouldTreatURLSchemeAsCORSEnabled(const String& scheme) {
DCHECK_EQ(scheme, scheme.LowerASCII());
if (scheme.IsEmpty())
......
......@@ -87,12 +87,6 @@ class PLATFORM_EXPORT SchemeRegistry {
static bool ShouldTreatURLSchemeAsNotAllowingJavascriptURLs(
const String& scheme);
// Allow non-HTTP schemes to be registered to allow CORS requests.
// This is not used in Chromium anymore but left here intentionally
// to allow other embedders of Blink to add more schemes
// to the CORS-enabled schemes list.
// As for now (Nov 2017) it is used by Electron.
static void RegisterURLSchemeAsCORSEnabled(const String& scheme);
static bool ShouldTreatURLSchemeAsCORSEnabled(const String& scheme);
// Serialize the registered schemes in a comma-separated list.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment