Fix handling of external protocols with PlzNavigate.

third_party/WebKit/Source/platform/weborigin/SchemeRegistry.cpp

 /*
  * Copyright (C) 2010 Apple Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 21 matching lines @@
 #include "wtf/ThreadingPrimitives.h"
 #include "wtf/text/StringBuilder.h"
 
 namespace blink {
 
 namespace {
 
 class URLSchemesRegistry final {
  public:
   URLSchemesRegistry()
-      : emptyDocumentSchemes({"about"}),
-        // For ServiceWorker schemes: HTTP is required because http://localhost
-        // is considered secure. Additional checks are performed to ensure that
-        // other http pages are filtered out.
+      :  // For ServiceWorker schemes: HTTP is required because http://localhost
+         // is considered secure. Additional checks are performed to ensure that
+         // other http pages are filtered out.
         serviceWorkerSchemes({"http", "https"}),
         fetchAPISchemes({"http", "https"}),
         allowedInReferrerSchemes({"http", "https"}) {
     for (auto& scheme : url::GetLocalSchemes())
       localSchemes.insert(scheme.c_str());
     for (auto& scheme : url::GetSecureSchemes())
       secureSchemes.insert(scheme.c_str());
     for (auto& scheme : url::GetNoAccessSchemes())
       schemesWithUniqueOrigins.insert(scheme.c_str());
     for (auto& scheme : url::GetCORSEnabledSchemes())
       CORSEnabledSchemes.insert(scheme.c_str());
     for (auto& scheme : url::GetCSPBypassingSchemes()) {
       contentSecurityPolicyBypassingSchemes.insert(
           scheme.c_str(), SchemeRegistry::PolicyAreaAll);
     }
+    for (auto& scheme : url::GetEmptyDocumentSchemes())
+      emptyDocumentSchemes.insert(scheme.c_str());
   }
   ~URLSchemesRegistry() = default;
 
   URLSchemesSet localSchemes;
   URLSchemesSet displayIsolatedURLSchemes;
   URLSchemesSet secureSchemes;
   URLSchemesSet schemesWithUniqueOrigins;
   URLSchemesSet emptyDocumentSchemes;
   URLSchemesSet schemesForbiddenFromDomainRelaxation;
   URLSchemesSet notAllowingJavascriptURLsSchemes;
@@ skipping 83 matching lines @@
   getMutableURLSchemesRegistry().secureSchemes.insert(scheme);
 }
 
 bool SchemeRegistry::shouldTreatURLSchemeAsSecure(const String& scheme) {
   DCHECK_EQ(scheme, scheme.lower());
   if (scheme.isEmpty())
     return false;
   return getURLSchemesRegistry().secureSchemes.contains(scheme);
 }
 
-void SchemeRegistry::registerURLSchemeAsEmptyDocument(const String& scheme) {
-  DCHECK_EQ(scheme, scheme.lower());
-  getMutableURLSchemesRegistry().emptyDocumentSchemes.insert(scheme);
-}
-
 bool SchemeRegistry::shouldLoadURLSchemeAsEmptyDocument(const String& scheme) {
   DCHECK_EQ(scheme, scheme.lower());
   if (scheme.isEmpty())
     return false;
   return getURLSchemesRegistry().emptyDocumentSchemes.contains(scheme);
 }
 
 void SchemeRegistry::setDomainRelaxationForbiddenForURLScheme(
     bool forbidden,
     const String& scheme) {
@@ skipping 185 matching lines @@
 
 bool SchemeRegistry::schemeShouldBypassSecureContextCheck(
     const String& scheme) {
   if (scheme.isEmpty())
     return false;
   DCHECK_EQ(scheme, scheme.lower());
   return getURLSchemesRegistry().secureContextBypassingSchemes.contains(scheme);
 }
 
 }  // namespace blink