JobController3: Move MarkAlternativeServiceBroken to job controller

net/http/http_stream_factory_impl_job_controller.h

Index: net/http/http_stream_factory_impl_job_controller.h
diff --git a/net/http/http_stream_factory_impl_job_controller.h b/net/http/http_stream_factory_impl_job_controller.h
index d54165028ab569ff72e50a2710da9e0e8703d539..e2004bdfb3668b0c8b6a13b69d0ae2939792cf05 100644
--- a/net/http/http_stream_factory_impl_job_controller.h
+++ b/net/http/http_stream_factory_impl_job_controller.h
@@ -201,6 +201,13 @@ class HttpStreamFactoryImpl::JobController
                            NextProto negotiated_protocol,
                            bool using_spdy);
 
+  // Must be called when |alternative_job_| fails.
+  void OnAlternativeJobFailed(Job* job);
+
+  // Called to report to http_server_properties to mark alternative service
+  // broken.
+  void ReportBrokenAlternativeService();
+
   void MaybeNotifyFactoryOfCompletion();
 
   // Called to resume the main job with delay.
@@ -255,6 +262,15 @@ class HttpStreamFactoryImpl::JobController
   std::unique_ptr<Job> main_job_;
   std::unique_ptr<Job> alternative_job_;
 
+  // True if |alternative_job_| uses alternative service/proxy server and it
+  // fails.
+  bool alternative_job_is_failed_;

[Ryan Hamilton, 2016/09/14 23:13:04]

nit: remove "_is"

[Zhongyi Shi, 2016/09/14 23:30:33]

Done.

+
+  // Either and only one of these records broken alternative service/proxy
+  // server that |alternative_job_| uses.
+  AlternativeService broken_alternative_service_;
+  ProxyServer broken_alternative_proxy_server_;

[Ryan Hamilton, 2016/09/14 23:13:04]

nit: can you use failed instead of broken here too? (both the variables and the
comments)

[Zhongyi Shi, 2016/09/14 23:30:33]

Done.

+
   // True if a Job has ever been bound to the |request_|.
   bool job_bound_;