Small fix for URLRequestJobs when entering suspending mode.

net/url_request/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 "net/url_request/url_request_job.h"
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
 #include "base/location.h"
 #include "base/metrics/histogram_macros.h"
@@ skipping 261 matching lines @@
 
 int URLRequestJob::GetResponseCode() const {
   return -1;
 }
 
 HostPortPair URLRequestJob::GetSocketAddress() const {
   return HostPortPair();
 }
 
 void URLRequestJob::OnSuspend() {
-  Kill();
+  // Most errors generated by the Job come as the result of the one current
+  // operation the job is waiting on returning an error. This event is unusual
+  // in that the Job may have another operation ongoing, or the Job may be idle
+  // and waiting on the next call.
+  //
+  // Need to cancel through the request to make sure everything is notified
+  // of the failure (Particularly that the NetworkDelegate, which the Job may be
+  // waiting on, is notified synchronously) and torn down correctly.
+  //
+  // TODO(mmenke): This should probably fail the request with
+  //               NETWORK_IO_SUSPENDED instead.
+  request_->Cancel();
 }
 
 void URLRequestJob::NotifyURLRequestDestroyed() {
 }
 
 void URLRequestJob::GetConnectionAttempts(ConnectionAttempts* out) const {
   out->clear();
 }
 
 // static
@@ skipping 647 matching lines @@
   int64_t total_received_bytes = GetTotalReceivedBytes();
   DCHECK_GE(total_received_bytes, last_notified_total_received_bytes_);
   if (total_received_bytes > last_notified_total_received_bytes_) {
     network_delegate_->NotifyNetworkBytesReceived(
         *request_, total_received_bytes - last_notified_total_received_bytes_);
   }
   last_notified_total_received_bytes_ = total_received_bytes;
 }
 
 }  // namespace net