Eliminate spurious request cancelations

content/child/resource_dispatcher.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.
 
 // See http://dev.chromium.org/developers/design-documents/multi-process-resource-loading
 
 #include "content/child/resource_dispatcher.h"
 
 #include <utility>
 
@@ skipping 471 matching lines @@
     if (should_dump) {
       char url_copy[256] = {0};
       strncpy(url_copy, info.response_url.spec().c_str(),
               sizeof(url_copy));
       base::debug::Alias(&url_copy);
       base::debug::Alias(&request_time);
       base::debug::DumpWithoutCrashing();
       should_dump = false;
     }
   }
-  // Cancel the request, and clean it up so the bridge will receive no more
-  // messages.
-  message_sender_->Send(new ResourceHostMsg_CancelRequest(request_id));
+  // Cancel the request if it didn't complete, and clean it up so the bridge
+  // will receive no more messages.
+  if (info.completion_time.is_null())
+    message_sender_->Send(new ResourceHostMsg_CancelRequest(request_id));
   RemovePendingRequest(request_id);
 }
 
 void ResourceDispatcher::SetDefersLoading(int request_id, bool value) {
   PendingRequestMap::iterator it = pending_requests_.find(request_id);
   if (it == pending_requests_.end()) {
     DLOG(ERROR) << "unknown request";
     return;
   }
   PendingRequestInfo* request_info = it->second.get();
@@ skipping 369 matching lines @@
     *frame_origin = extra_data->frame_origin();
   return request;
 }
 
 void ResourceDispatcher::SetResourceSchedulingFilter(
     scoped_refptr<ResourceSchedulingFilter> resource_scheduling_filter) {
   resource_scheduling_filter_ = resource_scheduling_filter;
 }
 
 }  // namespace content