Remove URLRequestJobFactory::IsHandledURL.

content/browser/loader/navigation_resource_throttle.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/browser/loader/navigation_resource_throttle.h"
 
 #include <memory>
 
 #include "base/bind.h"
 #include "base/callback.h"
@@ skipping 186 matching lines @@
   const ResourceRequestInfoImpl* info =
       ResourceRequestInfoImpl::ForRequest(request_);
   if (!info)
     return;
 
   int render_process_id, render_frame_id;
   if (!info->GetAssociatedRenderFrame(&render_process_id, &render_frame_id))
     return;
 
   bool is_external_protocol =
-      !info->GetContext()->GetRequestContext()->job_factory()->IsHandledURL(
-          request_->url());
+      request_->url().is_valid() &&
+      !info->GetContext()
+           ->GetRequestContext()
+           ->job_factory()
+           ->IsHandledProtocol(request_->url().scheme());

[mmenke, 2017/03/31 04:30:04]

This duplication seems ugly, but with PlzNavigate, should go away.  The one in
the NavigationResourceHandler will stay until we move that to mojo.

[asanka, 2017/03/31 20:38:30]

Acknowledged.

   UIChecksPerformedCallback callback =
       base::Bind(&NavigationResourceThrottle::OnUIChecksPerformed,
                  weak_ptr_factory_.GetWeakPtr());
   DCHECK(request_->method() == "POST" || request_->method() == "GET");
   BrowserThread::PostTask(
       BrowserThread::UI, FROM_HERE,
       base::Bind(&CheckWillStartRequestOnUIThread, callback, render_process_id,
                  render_frame_id, request_->method(), info->body(),
                  Referrer::SanitizeForRequest(
                      request_->url(), Referrer(GURL(request_->referrer()),
@@ skipping 13 matching lines @@
     return;
 
   if (redirect_info.new_method != "POST")
     info->ResetBody();
 
   int render_process_id, render_frame_id;
   if (!info->GetAssociatedRenderFrame(&render_process_id, &render_frame_id))
     return;
 
   bool new_is_external_protocol =
-      !info->GetContext()->GetRequestContext()->job_factory()->IsHandledURL(
-          request_->url());
+      request_->url().is_valid() &&
+      !info->GetContext()
+           ->GetRequestContext()
+           ->job_factory()
+           ->IsHandledProtocol(request_->url().scheme());
   DCHECK(redirect_info.new_method == "POST" ||
          redirect_info.new_method == "GET");
   UIChecksPerformedCallback callback =
       base::Bind(&NavigationResourceThrottle::OnUIChecksPerformed,
                  weak_ptr_factory_.GetWeakPtr());
 
   // Send the redirect info to the NavigationHandle on the UI thread.
   // Note: to avoid threading issues, a copy of the HttpResponseHeaders is sent
   // in lieu of the original.
   scoped_refptr<net::HttpResponseHeaders> response_headers;
@@ skipping 129 matching lines @@
 
   // If the results of the checks on the UI thread are known, unblock the
   // navigation. Otherwise, wait until the callback has executed.
   if (on_transfer_done_result_ != NavigationThrottle::DEFER) {
     OnUIChecksPerformed(on_transfer_done_result_);
     on_transfer_done_result_ = NavigationThrottle::DEFER;
   }
 }
 
 }  // namespace content