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

net/spdy/spdy_http_stream.h

 // Copyright (c) 2010 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.
 
 #ifndef NET_SPDY_SPDY_HTTP_STREAM_H_
 #define NET_SPDY_SPDY_HTTP_STREAM_H_
 #pragma once
 
 #include <list>
 #include <string>
@@ skipping 15 matching lines @@
 class IOBuffer;
 class SpdySession;
 class UploadData;
 class UploadDataStream;
 
 // The SpdyHttpStream is a HTTP-specific type of stream known to a SpdySession.
 class SpdyHttpStream : public SpdyStream::Delegate, public HttpStream {
  public:
   // SpdyHttpStream constructor
   SpdyHttpStream(SpdySession* spdy_session, bool direct);
+  // Constructs a SpdyHttpStream that wraps an existing SpdyStream.

[willchan no longer on Chromium, 2010/12/14 00:34:42]

Same style comment on overloading being disallowed.  In this case, I think you
might actually want to use a separate member function to set the stream.  It
would be a parallel of HttpStream::InitializeStream().  Maybe
InitializeWithExistingStream().

[Ryan Hamilton, 2010/12/15 20:14:17]

Done.

+  explicit SpdyHttpStream(SpdyStream* spdy_stream);
   virtual ~SpdyHttpStream();
 
   SpdyStream* stream() { return stream_.get(); }
 
   // ===================================================
   // HttpStream methods:
 
   // Initialize stream.  Must be called before calling SendRequest().
   virtual int InitializeStream(const HttpRequestInfo* request_info,
                                const BoundNetLog& net_log,
@@ skipping 134 matching lines @@
 
   // Is this spdy stream direct to the origin server (or to a proxy).
   bool direct_;
 
   DISALLOW_COPY_AND_ASSIGN(SpdyHttpStream);
 };
 
 }  // namespace net
 
 #endif  // NET_SPDY_SPDY_HTTP_STREAM_H_