From c1fd469d0f2e65c90b3dd94cf96d39a14aca0c74 Mon Sep 17 00:00:00 2001 From: klzgrad Date: Wed, 4 May 2022 14:54:28 +0800 Subject: [PATCH] cronet: Use fixed proxy resolution from experiment option proxy_server --- src/components/cronet/cronet_context.cc | 25 ++++++++++++++++--- .../cronet/cronet_global_state_stubs.cc | 6 ++--- 2 files changed, 24 insertions(+), 7 deletions(-) diff --git a/src/components/cronet/cronet_context.cc b/src/components/cronet/cronet_context.cc index 1fa1e93712..fc3f38bef5 100644 --- a/src/components/cronet/cronet_context.cc +++ b/src/components/cronet/cronet_context.cc @@ -12,6 +12,7 @@ #include #include #include +#include #include #include "base/base64.h" @@ -53,7 +54,10 @@ #include "net/log/net_log_util.h" #include "net/net_buildflags.h" #include "net/nqe/network_quality_estimator_params.h" +#include "net/proxy_resolution/configured_proxy_resolution_service.h" +#include "net/proxy_resolution/proxy_config.h" #include "net/proxy_resolution/proxy_config_service_fixed.h" +#include "net/proxy_resolution/proxy_config_with_annotation.h" #include "net/proxy_resolution/proxy_resolution_service.h" #include "net/third_party/quiche/src/quic/core/quic_versions.h" #include "net/url_request/url_request_context.h" @@ -352,9 +356,24 @@ CronetContext::NetworkTasks::BuildDefaultURLRequestContext( net::URLRequestContextBuilder context_builder; SetSharedURLRequestContextBuilderConfig(&context_builder); - context_builder.set_proxy_resolution_service( - cronet::CreateProxyResolutionService(std::move(proxy_config_service), - g_net_log.Get().net_log())); + const auto proxy_server_it = + context_config_->experimental_options.find("proxy_server"); + std::string proxy_server_str = "direct://"; + if (proxy_server_it != context_config_->experimental_options.end()) { + const base::Value& value = proxy_server_it->second; + if (value.is_string()) { + proxy_server_str = value.GetString(); + } + } + net::ProxyConfig proxy_config; + proxy_config.proxy_rules().ParseFromString(proxy_server_str); + auto proxy_service = + net::ConfiguredProxyResolutionService::CreateWithoutProxyResolver( + std::make_unique( + net::ProxyConfigWithAnnotation(proxy_config, + MISSING_TRAFFIC_ANNOTATION)), + g_net_log.Get().net_log()); + context_builder.set_proxy_resolution_service(std::move(proxy_service)); if (context_config_->enable_network_quality_estimator) { std::unique_ptr nqe_params = diff --git a/src/components/cronet/cronet_global_state_stubs.cc b/src/components/cronet/cronet_global_state_stubs.cc index 9ff771b7b3..6bb5b2b673 100644 --- a/src/components/cronet/cronet_global_state_stubs.cc +++ b/src/components/cronet/cronet_global_state_stubs.cc @@ -63,15 +63,13 @@ void PostTaskToInitThread(const base::Location& posted_from, std::unique_ptr CreateProxyConfigService( const scoped_refptr& io_task_runner) { - return net::ConfiguredProxyResolutionService::CreateSystemProxyConfigService( - io_task_runner); + return nullptr; } std::unique_ptr CreateProxyResolutionService( std::unique_ptr proxy_config_service, net::NetLog* net_log) { - return net::ConfiguredProxyResolutionService::CreateUsingSystemProxyResolver( - std::move(proxy_config_service), net_log, /*quick_check_enabled=*/true); + return nullptr; } std::string CreateDefaultUserAgent(const std::string& partial_user_agent) {