Failure in reading Blob URL should results in network error

storage/browser/blob/blob_url_request_job.cc

 // Copyright (c) 2012 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 "storage/browser/blob/blob_url_request_job.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <limits>
@@ skipping 173 matching lines @@
     if (!blob_handle->content_disposition().empty()) {
       std::string content_disposition_header("Content-Disposition: ");
       content_disposition_header.append(blob_handle->content_disposition());
       headers->AddHeader(content_disposition_header);
     }
   }
 
   return headers;
 }
 
-net::HttpStatusCode BlobURLRequestJob::NetErrorToHttpStatusCode(
-    int error_code) {
-  net::HttpStatusCode status_code = net::HTTP_INTERNAL_SERVER_ERROR;
-  switch (error_code) {
-    case net::ERR_ACCESS_DENIED:
-      status_code = net::HTTP_FORBIDDEN;
-      break;
-    case net::ERR_FILE_NOT_FOUND:
-      status_code = net::HTTP_NOT_FOUND;
-      break;
-    case net::ERR_METHOD_NOT_SUPPORTED:
-      status_code = net::HTTP_METHOD_NOT_ALLOWED;
-      break;
-    case net::ERR_REQUEST_RANGE_NOT_SATISFIABLE:
-      status_code = net::HTTP_REQUESTED_RANGE_NOT_SATISFIABLE;
-      break;
-    case net::ERR_INVALID_ARGUMENT:
-      status_code = net::HTTP_BAD_REQUEST;
-      break;
-    case net::ERR_CACHE_READ_FAILURE:
-    case net::ERR_CACHE_CHECKSUM_READ_FAILURE:
-    case net::ERR_UNEXPECTED:
-    case net::ERR_FAILED:
-      break;
-    default:
-      DCHECK(false) << "Error code not supported: " << error_code;
-      break;
-  }
-  return status_code;
-}
-
 BlobURLRequestJob::~BlobURLRequestJob() {
   TRACE_EVENT_ASYNC_END1("Blob", "BlobRequest", this, "uuid",
                          blob_handle_ ? blob_handle_->uuid() : "NotFound");
 }
 
 void BlobURLRequestJob::DidStart() {
   error_ = false;
 
   // We only support GET request per the spec.
   if (request()->method() != "GET") {
@@ skipping 81 matching lines @@
   ReadRawDataComplete(result);
 }
 
 void BlobURLRequestJob::NotifyFailure(int error_code) {
   error_ = true;
 
   // If we already return the headers on success, we can't change the headers
   // now. Instead, we just error out.
   DCHECK(!response_info_) << "Cannot NotifyFailure after headers.";
 
-  HeadersCompleted(NetErrorToHttpStatusCode(error_code));
+  NotifyStartError(net::URLRequestStatus::FromError(error_code));
 }
 
 void BlobURLRequestJob::HeadersCompleted(net::HttpStatusCode status_code) {
   int64_t content_size = 0;
   response_info_.reset(new net::HttpResponseInfo());
   response_info_->headers =
       GenerateHeaders(status_code, blob_handle_.get(), blob_reader_.get(),
                       &byte_range_, &content_size);
   set_expected_content_size(content_size);
   if (blob_reader_)
     response_info_->metadata = blob_reader_->side_data();
 
   NotifyHeadersComplete();
 }
 
 }  // namespace storage