Make alternate Jobs not use HTTP/1.1 sockets.

net/http/http_stream_factory_impl_job.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 "net/http/http_stream_factory_impl_job.h"
 
 #include <algorithm>
 #include <string>
 
 #include "base/bind.h"
@@ skipping 1089 matching lines @@
 
   next_state_ = STATE_CREATE_STREAM_COMPLETE;
 
   // We only set the socket motivation if we're the first to use
   // this socket.  Is there a race for two SPDY requests?  We really
   // need to plumb this through to the connect level.
   if (connection_->socket() && !connection_->is_reused())
     SetSocketMotivation();
 
   if (!using_spdy_) {
+    if (IsAlternate())

[Ryan Hamilton, 2015/05/05 18:31:55]

nit: I think this should be IsSpdyAlternate(), since that's the only type of
AlternateJob which does NPN at all.

You could also do something like this up at the start of the method.

if (IsAlternate()) 
  DCHECK(IsSpdyAlternate())

to document that only SPDY alternate jobs hit this codepath (and that QUIC jobs
do not).

[Bence, 2015/05/05 20:14:18]

Done.

BTW I moved the check from here to DoInitConnectionComplete.
Done.

+      return ERR_NPN_NEGOTIATION_FAILED;
     // We may get ftp scheme when fetching ftp resources through proxy.
     bool using_proxy = (proxy_info_.is_http() || proxy_info_.is_https()) &&
                        (request_info_.url.SchemeIs("http") ||
                         request_info_.url.SchemeIs("ftp"));
     if (stream_factory_->for_websockets_) {
       DCHECK(request_);
       DCHECK(request_->websocket_handshake_stream_create_helper());
       websocket_stream_.reset(
           request_->websocket_handshake_stream_create_helper()
               ->CreateBasicStream(connection_.Pass(), using_proxy));
@@ skipping 426 matching lines @@
 
 void HttpStreamFactoryImpl::Job::
     MaybeCopyConnectionAttemptsFromClientSocketHandleToRequest() {
   if (IsOrphaned() || !connection_)
     return;
 
   request_->AddConnectionAttempts(connection_->connection_attempts());
 }
 
 }  // namespace net