cronet: Use fixed proxy resolution from experimental option proxy_server

This commit is contained in:
klzgrad 2022-05-04 14:54:28 +08:00
parent 5811073952
commit 69c8765c03
3 changed files with 29 additions and 7 deletions

View File

@ -12,6 +12,7 @@
#include <map>
#include <memory>
#include <set>
#include <string>
#include <utility>
#include "base/base64.h"
@ -52,7 +53,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/quiche/quic/core/quic_versions.h"
#include "net/url_request/url_request_context.h"
@ -356,9 +360,21 @@ CronetContext::NetworkTasks::BuildDefaultURLRequestContext(
context_config_->ConfigureURLRequestContextBuilder(&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 base::Value* proxy_server =
context_config_->effective_experimental_options.Find("proxy_server");
std::string proxy_server_str = "direct://";
if (proxy_server != nullptr && proxy_server->is_string()) {
proxy_server_str = proxy_server->GetString();
}
net::ProxyConfig proxy_config;
proxy_config.proxy_rules().ParseFromString(proxy_server_str);
auto proxy_service =
net::ConfiguredProxyResolutionService::CreateWithoutProxyResolver(
std::make_unique<net::ProxyConfigServiceFixed>(
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<net::NetworkQualityEstimatorParams> nqe_params =

View File

@ -62,15 +62,13 @@ void PostTaskToInitThread(const base::Location& posted_from,
std::unique_ptr<net::ProxyConfigService> CreateProxyConfigService(
const scoped_refptr<base::SequencedTaskRunner>& io_task_runner) {
return net::ProxyConfigService::CreateSystemProxyConfigService(
io_task_runner);
return nullptr;
}
std::unique_ptr<net::ProxyResolutionService> CreateProxyResolutionService(
std::unique_ptr<net::ProxyConfigService> 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) {

View File

@ -783,6 +783,14 @@ void URLRequestContextConfig::SetContextBuilderExperimentalOptions(
continue;
}
session_params->spdy_go_away_on_ip_change = iter->second.GetBool();
} else if (iter->first == "proxy_server") {
if (!iter->second.is_string()) {
LOG(ERROR) << "\"" << iter->first << "\" config params \""
<< iter->second << "\" is not a string";
effective_experimental_options.Remove(iter->first);
continue;
}
// Handled in CronetContext::NetworkTasks::BuildDefaultURLRequestContext.
} else {
LOG(WARNING) << "Unrecognized Cronet experimental option \""
<< iter->first << "\" with params \"" << iter->second;