Fixes crash at ResourceDispatcherHost::BeginDownload...

chrome/browser/resource_dispatcher_host.cc

 // Copyright (c) 2006-2008 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.
 
 // See http://wiki.corp.google.com/twiki/bin/view/Main/ChromeMultiProcessResourceLoading
 
 #include <vector>
 
 #include "chrome/browser/resource_dispatcher_host.h"
 
@@ skipping 1591 matching lines @@
 
 // We are explicitly forcing the download of 'url'.
 void ResourceDispatcherHost::BeginDownload(const GURL& url,
                                            const GURL& referrer,
                                            int render_process_host_id,
                                            int render_view_id,
                                            URLRequestContext* request_context) {
   if (is_shutdown_)
     return;
 
+  if (!URLRequest::IsHandledURL(url))
+    return;
+
   // Check if the renderer is permitted to request the requested URL.
   //
   // TODO(mpcomplete): remove "render_process_host_id != -1"
   //                   when http://b/viewIssue?id=1080959 is fixed.
   if (render_process_host_id != -1 &&
       !RendererSecurityPolicy::GetInstance()->
           CanRequestURL(render_process_host_id, url)) {
     LOG(INFO) << "Denied unauthorized download request for " <<
         url.possibly_invalid_spec();
     return;
@@ skipping 20 matching lines @@
   if (safe_browsing_->enabled() && safe_browsing_->CanCheckUrl(url)) {
     handler = new SafeBrowsingEventHandler(handler,
                                            render_process_host_id,
                                            render_view_id,
                                            url,
                                            ResourceType::MAIN_FRAME,
                                            safe_browsing_,
                                            this);
   }
 
-  bool known_proto = URLRequest::IsHandledURL(url);
-  if (!known_proto) {
-    CHECK(false);
-  }
-
   request->set_method("GET");
   request->set_referrer(referrer.spec());
   request->set_context(request_context);
 
   ExtraRequestInfo* extra_info =
       new ExtraRequestInfo(handler,
                            request_id_,
                            render_process_host_id,
                            render_view_id,
                            false,  // Downloads are not considered mixed-content
@@ skipping 843 matching lines @@
   bool enough_new_progress = (amt_since_last > (size / kHalfPercentIncrements));
   bool too_much_time_passed = time_since_last > kOneSecond;
 
   if (is_finished || enough_new_progress || too_much_time_passed) {
     info->event_handler->OnUploadProgress(info->request_id, position, size);
     info->waiting_for_upload_progress_ack = true;
     info->last_upload_ticks = TimeTicks::Now();
     info->last_upload_position = position;
   }
 }