Remove URLRequest::IsHandledProtocol and IsHandledURL

chromecast/browser/cast_content_browser_client.cc

 // Copyright 2014 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 "chromecast/browser/cast_content_browser_client.h"
 
 #include <stddef.h>
 
 #include <string>
 #include <utility>
@@ skipping 178 matching lines @@
       new network_hints::NetworkHintsMessageFilter(
           url_request_context_factory_->host_resolver()));
   host->AddFilter(network_hints_message_filter.get());
 }
 
 bool CastContentBrowserClient::IsHandledURL(const GURL& url) {
   if (!url.is_valid())
     return false;
 
   static const char* const kProtocolList[] = {
-    content::kChromeUIScheme,
-    content::kChromeDevToolsScheme,
-    kChromeResourceScheme,
-    url::kBlobScheme,
-    url::kDataScheme,
-    url::kFileSystemScheme,
+      content::kChromeUIScheme, content::kChromeDevToolsScheme,
+      kChromeResourceScheme,    url::kBlobScheme,
+      url::kDataScheme,         url::kFileSystemScheme,
+      url::kHttpScheme,         url::kHttpsScheme,
+      url::kWsScheme,           url::kWssScheme,
   };
 
   const std::string& scheme = url.scheme();
   for (size_t i = 0; i < arraysize(kProtocolList); ++i) {
     if (scheme == kProtocolList[i])
       return true;
   }
 
   if (scheme == url::kFileScheme) {
     return base::CommandLine::ForCurrentProcess()->HasSwitch(
         switches::kEnableLocalFileAccesses);
   }
 
   return false;

[mmenke, 2016/04/21 18:24:19]

I guess this didn't noticeably break anything because of the "&&
!net::URLRequest::IsHandledURL(url)"
ChildProcessSecurityPolicyImpl::CanRequestURL?

[mgersh, 2016/04/21 19:04:45]

Looks like it. I guess it's never been called anywhere that doesn't also call
URLRequest::IsHandledURL().

 }
 
 void CastContentBrowserClient::AppendExtraCommandLineSwitches(
     base::CommandLine* command_line,
     int child_process_id) {
   std::string process_type =
       command_line->GetSwitchValueNative(switches::kProcessType);
   base::CommandLine* browser_command_line =
       base::CommandLine::ForCurrentProcess();
 
@@ skipping 262 matching lines @@
         process_type, dumps_path, false /* upload */);
   // StartUploaderThread() even though upload is diferred.
   // Breakpad-related memory is freed in the uploader thread.
   crash_handler->StartUploaderThread();
   return crash_handler;
 }
 #endif  // !defined(OS_ANDROID)
 
 }  // namespace shell
 }  // namespace chromecast