net: remove forced renegotiation checks

net/http/http_stream_factory_impl_job.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/http/http_stream_factory_impl_job.h"
 
 #include "base/logging.h"
 #include "base/stl_util-inl.h"
 #include "base/string_util.h"
 #include "base/stringprintf.h"
@@ skipping 868 matching lines @@
     // client certificate during the initial handshake.
     // http://crbug.com/59292
     ssl_config->false_start_enabled = false;
   }
 
   UMA_HISTOGRAM_ENUMERATION("Net.ConnectionUsedSSLv3Fallback",
                             static_cast<int>(ssl_config->ssl3_fallback), 2);
 
   if (request_info_.load_flags & LOAD_VERIFY_EV_CERT)
     ssl_config->verify_ev_cert = true;
-
-  if (proxy_info_.proxy_server().scheme() == ProxyServer::SCHEME_HTTP ||
-      proxy_info_.proxy_server().scheme() == ProxyServer::SCHEME_HTTPS) {
-    ssl_config->mitm_proxies_allowed = true;
-  }

[wtc, 2011/04/05 17:20:03]

Given this code to allow MITM proxies whenever a proxy is
used, it seems that the problem is with anti-virus software
or proxies that the browser is not aware of?

[agl, 2011/04/05 19:55:41]

Indeed.

 }
 
 
 int HttpStreamFactoryImpl::Job::ReconsiderProxyAfterError(int error) {
   DCHECK(!pac_request_);
 
   // A failure to resolve the hostname or any error related to establishing a
   // TCP connection could be grounds for trying a new proxy configuration.
   //
   // Why do this when a hostname cannot be resolved?  Some URLs only make sense
@@ skipping 127 matching lines @@
 bool HttpStreamFactoryImpl::Job::IsPreconnecting() const {
   DCHECK_GE(num_streams_, 0);
   return num_streams_ > 0;
 }
 
 bool HttpStreamFactoryImpl::Job::IsOrphaned() const {
   return !IsPreconnecting() && !request_;
 }
 
 }  // namespace net