Fix logic for handling reports of malformed bodies. To end up counting

net/url_request/url_request_throttler_entry.cc

 // Copyright (c) 2011 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_throttler_entry.h"
 
 #include <cmath>
 
 #include "base/logging.h"
 #include "base/rand_util.h"
@@ skipping 178 matching lines @@
       HandleCustomRetryAfter(retry_header);
 
     std::string throttling_header = response->GetNormalizedValue(
         kExponentialThrottlingHeader);
     if (!throttling_header.empty())
       HandleThrottlingHeader(throttling_header, host);
   }
 }
 
 void URLRequestThrottlerEntry::ReceivedContentWasMalformed() {
-  // We keep this simple and just count it as a single error.
-  //
-  // If we wanted to get fancy, we would count two errors here, and decrease
-  // the error count only by one when we receive a successful (by status
-  // code) response. Instead, we keep things simple by always resetting the
-  // error count on success, and therefore counting only a single error here.
+  // A malformed body can only occur when the request to fetch a resource
+  // was successful.  Therefore, in such a situation, we will receive one
+  // call to ReceivedContentWasMalformed() and one call to UpdateWithResponse()
+  // with a response categorized as "good".  To end up counting one failure,
+  // we need to count two failures here against the one success in
+  // UpdateWithResponse().
+  GetBackoffEntry()->InformOfRequest(false);
   GetBackoffEntry()->InformOfRequest(false);
 }
 
 URLRequestThrottlerEntry::~URLRequestThrottlerEntry() {
 }
 
 void URLRequestThrottlerEntry::Initialize() {
   sliding_window_release_time_ = base::TimeTicks::Now();
   backoff_policy_.num_errors_to_ignore = kDefaultNumErrorsToIgnore;
   backoff_policy_.initial_backoff_ms = kDefaultInitialBackoffMs;
@@ skipping 43 matching lines @@
 
 const BackoffEntry* URLRequestThrottlerEntry::GetBackoffEntry() const {
   return &backoff_entry_;
 }
 
 BackoffEntry* URLRequestThrottlerEntry::GetBackoffEntry() {
   return &backoff_entry_;
 }
 
 }  // namespace net