Report ERR_CONTENT_LENGTH_MISMATCH when the count of bytes received doesn't match Content-Length.

webkit/glue/weburlloader_impl.cc

Index: webkit/glue/weburlloader_impl.cc
diff --git a/webkit/glue/weburlloader_impl.cc b/webkit/glue/weburlloader_impl.cc
index 7b4ec9b79f564571687f4697015ead4f85ba40e7..88f4565009836ff076de2bda56ad43d4addd2eea 100644
--- a/webkit/glue/weburlloader_impl.cc
+++ b/webkit/glue/weburlloader_impl.cc
@@ -592,7 +592,7 @@ void WebURLLoaderImpl::Context::OnReceivedCachedMetadata(
 }
 
 void WebURLLoaderImpl::Context::OnCompletedRequest(
-    const net::URLRequestStatus& status,
+    const net::URLRequestStatus& original_status,
     const std::string& security_info,
     const base::Time& completion_time) {
   if (ftp_listing_delegate_.get()) {
@@ -603,6 +603,14 @@ void WebURLLoaderImpl::Context::OnCompletedRequest(
     multipart_delegate_.reset(NULL);
   }
 
+  net::URLRequestStatus status = original_status;
+
+  // Rewrite the Content-Length mismatch as a success.
+  // See crbug.com/52847 for justification.
+  if (status.status() != net::URLRequestStatus::SUCCESS &&
+      status.error() == net::ERR_CONTENT_LENGTH_MISMATCH)
+    status.set_status(net::URLRequestStatus::SUCCESS);

[darin (slow to review), 2011/11/16 21:59:57]

should you also clear the error field here for completeness?

[cbentzel, 2011/11/16 22:19:28]

Done. The code below doesn't investigate error if status is SUCCESS, but that
may drift over time. This almost seems like it should be an invariant for
URLRequestStatus.

+
   // Prevent any further IPC to the browser now that we're complete, but
   // don't delete it to keep any downloaded temp files alive.
   DCHECK(!completed_bridge_.get());