ABANDONED Handle multiple AlternateProtocols for each HostPortPair.

net/http/http_stream_factory_impl_job.h

 // 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.
 
 #ifndef NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_
 
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/memory/weak_ptr.h"
@@ skipping 257 matching lines @@
   HostPortPair origin_;
 
   // The origin url we're trying to reach. This url may be different from the
   // original request when host mapping rules are set-up.
   GURL origin_url_;
 
   // If this is a Job for an "Alternate-Protocol", then this will be non-NULL
   // and will specify the original URL.
   scoped_ptr<GURL> original_url_;
 
+  // If this is a Job for an "Alternate-Protocol", then this will specify the
+  // alternate protocol.
+  AlternateProtocolInfo alternate_protocol_;

[Ryan Hamilton, 2014/11/03 19:00:31]

I think this should probably be initialized in the constructor.

[Bence, 2014/11/03 22:32:26]

I specifically added a default constructor to AlternateProtocolInfo that would
set it to UNINITIALIZED_ALTERNATE_PROTOCOL, is that not acceptable?

[Ryan Hamilton, 2014/11/04 18:56:22]

Ah, good point. For some reason I was thinking it was simply an enum.

+
   // This is the Job we're dependent on. It will notify us if/when it's OK to
   // proceed.
   Job* blocking_job_;
 
   // |waiting_job_| is a Job waiting to see if |this| can reuse a connection.
   // If |this| is unable to do so, we'll notify |waiting_job_| that it's ok to
   // proceed and then race the two Jobs.
   Job* waiting_job_;
 
   // True if handling a HTTPS request, or using SPDY with SSL
@@ skipping 48 matching lines @@
   JobStatus other_job_status_;
 
   base::WeakPtrFactory<Job> ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(Job);
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_H_