Correctly create a new stream when the server requires authentication.

net/http/http_pipelined_stream.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_pipelined_stream.h"
 
 #include "base/logging.h"
 #include "base/stringprintf.h"
 #include "net/base/net_errors.h"
 #include "net/http/http_pipelined_connection_impl.h"
@@ skipping 53 matching lines @@
 int HttpPipelinedStream::ReadResponseBody(IOBuffer* buf, int buf_len,
                                           OldCompletionCallback* callback) {
   return pipeline_->ReadResponseBody(pipeline_id_, buf, buf_len, callback);
 }
 
 void HttpPipelinedStream::Close(bool not_reusable) {
   pipeline_->Close(pipeline_id_, not_reusable);
 }
 
 HttpStream* HttpPipelinedStream::RenewStreamForAuth() {
-  // FIXME: What does this mean on a pipeline? Is it for proxies?
-  return new HttpPipelinedStream(pipeline_, pipeline_id_);
+  if (pipeline_->usable()) {
+    return pipeline_->CreateNewStream();
+  } else {

[mmenke, 2011/11/01 14:21:55]

nit:  It's chrome style not to use else statements in this case.

+    return NULL;
+  }
 }
 
 bool HttpPipelinedStream::IsResponseBodyComplete() const {
   return pipeline_->IsResponseBodyComplete(pipeline_id_);
 }
 
 bool HttpPipelinedStream::CanFindEndOfResponse() const {
   return pipeline_->CanFindEndOfResponse(pipeline_id_);
 }
 
@@ skipping 47 matching lines @@
 
 const NetLog::Source& HttpPipelinedStream::source() const {
   return pipeline_->source();
 }
 
 bool HttpPipelinedStream::was_npn_negotiated() const {
   return pipeline_->was_npn_negotiated();
 }
 
 }  // namespace net