Make HttpNetworkSession::host_mapping_rules no longer a pointer.

net/http/http_stream_factory_impl_job_controller.cc

Index: net/http/http_stream_factory_impl_job_controller.cc
diff --git a/net/http/http_stream_factory_impl_job_controller.cc b/net/http/http_stream_factory_impl_job_controller.cc
index 1e7ea76ade0fcbbf8bf9106caa33ab44ac0352c3..51004d0da349ff3cfdebc58c2fc8a92f6381770e 100644
--- a/net/http/http_stream_factory_impl_job_controller.cc
+++ b/net/http/http_stream_factory_impl_job_controller.cc
@@ -894,8 +894,9 @@ void HttpStreamFactoryImpl::JobController::MaybeNotifyFactoryOfCompletion() {
 GURL HttpStreamFactoryImpl::JobController::ApplyHostMappingRules(
     const GURL& url,
     HostPortPair* endpoint) {
-  const HostMappingRules* mapping_rules = session_->params().host_mapping_rules;
-  if (mapping_rules && mapping_rules->RewriteHost(endpoint)) {
+  const HostMappingRules* mapping_rules =
+      &session_->params().host_mapping_rules;
+  if (mapping_rules->RewriteHost(endpoint)) {

[Randy Smith (Not in Mondays), 2017/05/25 17:50:48]

nit, suggestion: This might all fit on one line without the extra variable. 
Less usefully, the variable could be a reference (I'm not sure that's even a
suggestion, just an alternative.)

[mmenke, 2017/05/25 18:07:10]

Done.

     url::Replacements<char> replacements;
     const std::string port_str = base::UintToString(endpoint->port());
     replacements.SetPort(port_str.c_str(), url::Component(0, port_str.size()));