Allow a non-200 (or non-407) response for a CONNECT request from an HTTPS pro...

net/http/stream_factory.h

Index: net/http/stream_factory.h
===================================================================
--- net/http/stream_factory.h	(revision 68026)
+++ net/http/stream_factory.h	(working copy)
@@ -69,6 +69,14 @@
     // may take a reference if it needs the cert_info beyond the lifetime of
     // this callback.
     virtual void OnNeedsClientAuth(SSLCertRequestInfo* cert_info) = 0;
+
+    // This is the failure of the CONNECT request through an HTTPS proxy.
+    // Headers can be read from |response_info|, while the body can be read
+    // from |stream|.
+    // Ownership of |response_info| is retained by the StreamRequest.

[vandebo (ex-Chrome), 2010/12/04 00:30:37]

I don't think you need to note the ownership of a const class ref

[Ryan Hamilton, 2010/12/09 21:19:35]

Oh, duh!

+    // Ownership of |stream| is transferred to the delegate.
+    virtual void OnHttpsProxyTunnelConnectionResponse(

[vandebo (ex-Chrome), 2010/12/04 00:30:37]

Connection -> Connect ?

+        const HttpResponseInfo& response_info, HttpStream* stream) = 0;
   };
 
   virtual ~StreamRequest() {}