JobController3: Move MarkAlternativeServiceBroken to job controller

net/http/http_stream_factory_impl_job_controller.h

 // Copyright (c) 2016 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_CONTROLLER_H_
 #define NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_CONTROLLER_H_
 
 #include "net/base/host_port_pair.h"
 #include "net/http/http_stream_factory_impl_job.h"
 #include "net/http/http_stream_factory_impl_request.h"
@@ skipping 48 matching lines @@
   void OnRequestComplete() override;
 
   // Called to resume the HttpStream creation process when necessary
   // Proxy authentication credentials are collected.
   int RestartTunnelWithProxyAuth(const AuthCredentials& credentials) override;
 
   // Called when the priority of transaction changes.
   void SetPriority(RequestPriority priority) override;
 
   // From HttpStreamFactoryImpl::Job::Delegate.
+
+  // Invoked when |job| has alternative service identified borken.
+  void OnAlternativeServiceBroken(Job* job) override;
+
+  // Invoked when |job| has alternative proxy server identified broken.
+  void OnAlternativeProxyServerBroken(Job* job) override;
+
   // Invoked when |job| has an HttpStream ready.
   void OnStreamReady(Job* job,
                      const SSLConfig& used_ssl_config,
                      const ProxyInfo& used_proxy_info) override;
 
   // Invoked when |job| has a BidirectionalStream ready.
   void OnBidirectionalStreamImplReady(
       Job* job,
       const SSLConfig& used_ssl_config,
       const ProxyInfo& used_proxy_info) override;
@@ skipping 115 matching lines @@
   void OrphanUnboundJob();
 
   // Called when a Job succeeds.
   void OnJobSucceeded(Job* job);
 
   // Marks completion of the |request_|.
   void MarkRequestComplete(bool was_npn_negotiated,
                            NextProto negotiated_protocol,
                            bool using_spdy);
 
+  // Only be called in successful cases when a job manages to serve the request.
+  void MaybeMarkAlternativeServiceBroken();
+
   void MaybeNotifyFactoryOfCompletion();
 
   // Called to resume the main job with delay.
   void MaybeResumeMainJob(Job* job, const base::TimeDelta& delay);
 
   void ResumeMainJob();
 
   // Returns true if QUIC is whitelisted for |host|.
   bool IsQuicWhitelistedForHost(const std::string& host);
 
@@ skipping 34 matching lines @@
 
   // True if this JobController is used to preconnect streams.
   bool is_preconnect_;
 
   // |main_job_| is a job waiting to see if |alternative_job_| can reuse a
   // connection. If |alternative_job_| is unable to do so, |this| will notify
   // |main_job_| to proceed and then race the two jobs.
   std::unique_ptr<Job> main_job_;
   std::unique_ptr<Job> alternative_job_;
 
+  // True if |alternative_job_| uses alternative service and discovers it as
+  // broken.
+  bool alternative_service_is_broken_;

[tbansal1, 2016/09/13 05:06:45]

Is it possible to remove the bool variable since 
alternative_service_is_broken_ should be true iff broken_alternative_service_ is
a valid alternative service.

Same comment for the bool alternative_proxy_server_is_broken_ below. 

+  AlternativeService broken_alternative_service_;
+  // True if |alternative_job_| uses alternative proxy server and discovers it
+  // as broken or failed.
+  bool alternative_proxy_server_is_broken_;
+  ProxyServer broken_alternative_proxy_server_;
+
   // True if a Job has ever been bound to the |request_|.
   bool job_bound_;
 
   // True if the main job has to wait for the alternative job: i.e., the main
   // job must not create a connection until it is resumed.
   bool main_job_is_blocked_;
   // Waiting time for the main job before it is resumed.
   base::TimeDelta main_job_wait_time_;
 
   // At the point where a Job is irrevocably tied to |request_|, we set this.
   // It will be nulled when the |request_| is finished.
   Job* bound_job_;
 
   // True if an alternative proxy server job can be started to fetch |request_|.
   bool can_start_alternative_proxy_job_;
 
   base::WeakPtrFactory<JobController> ptr_factory_;
 };
 
 }  // namespace net
 
 #endif  // NET_HTTP_HTTP_STREAM_FACTORY_IMPL_JOB_CONTROLLER_H_