Tighten the check for detecting misbehaving proxies in device management requests.

chrome/browser/policy/device_management_service.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 "chrome/browser/policy/device_management_service.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/compiler_specific.h"
@@ skipping 522 matching lines @@
   // Retry the job if it failed due to a broken proxy, by bypassing the
   // proxy on the next try. Don't retry if this URLFetcher already bypassed
   // the proxy.
   bool retry = false;
   if ((source->GetLoadFlags() & net::LOAD_BYPASS_PROXY) == 0) {
     if (!source->GetStatus().is_success() &&
         IsProxyError(source->GetStatus())) {
       LOG(WARNING) << "Proxy failed while contacting dmserver.";
       retry = true;
     } else if (source->GetStatus().is_success() &&
+               source->GetResponseCode() == kSuccess &&
                source->WasFetchedViaProxy() &&
                !IsProtobufMimeType(source)) {
       // The proxy server can be misconfigured but pointing to an existing
       // server that replies to requests. Try to recover if a successful
       // request that went through a proxy returns an unexpected mime type.
       LOG(WARNING) << "Got bad mime-type in response from dmserver that was "
                    << "fetched via a proxy.";
       retry = true;
     }
   }
@@ skipping 28 matching lines @@
     }
   }
 
   const JobQueue::iterator elem =
       std::find(queued_jobs_.begin(), queued_jobs_.end(), job);
   if (elem != queued_jobs_.end())
     queued_jobs_.erase(elem);
 }
 
 }  // namespace policy