Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(107)

Unified Diff: components/data_reduction_proxy/core/common/data_reduction_proxy_event_creator.cc

Issue 1871783002: Remove the Data Reduction Proxy TLS experiment code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: tbansal comments Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/data_reduction_proxy/core/common/data_reduction_proxy_event_creator.cc
diff --git a/components/data_reduction_proxy/core/common/data_reduction_proxy_event_creator.cc b/components/data_reduction_proxy/core/common/data_reduction_proxy_event_creator.cc
index f7d418b21fa4e88593875ac18f6c95859ac86143..4fd298b16cc4beb3dc7c357a115e139ed57517e8 100644
--- a/components/data_reduction_proxy/core/common/data_reduction_proxy_event_creator.cc
+++ b/components/data_reduction_proxy/core/common/data_reduction_proxy_event_creator.cc
@@ -42,7 +42,6 @@ int64_t GetExpirationTicks(int bypass_seconds) {
scoped_ptr<base::Value> EnableDataReductionProxyCallback(
bool secure_transport_restricted,
const std::vector<net::ProxyServer>& proxies_for_http,
- const std::vector<net::ProxyServer>& proxies_for_https,
net::NetLogCaptureMode /* capture_mode */) {
scoped_ptr<base::DictionaryValue> dict(new base::DictionaryValue());
dict->SetBoolean("enabled", true);
@@ -51,12 +50,7 @@ scoped_ptr<base::Value> EnableDataReductionProxyCallback(
for (const auto& proxy : proxies_for_http)
http_proxy_list->AppendString(proxy.ToURI());
- scoped_ptr<base::ListValue> https_proxy_list(new base::ListValue());
- for (const auto& proxy : proxies_for_https)
- https_proxy_list->AppendString(proxy.ToURI());
-
dict->Set("http_proxy_list", std::move(http_proxy_list));
- dict->Set("https_proxy_list", std::move(https_proxy_list));
return std::move(dict);
}
@@ -178,12 +172,11 @@ DataReductionProxyEventCreator::~DataReductionProxyEventCreator() {
void DataReductionProxyEventCreator::AddProxyEnabledEvent(
net::NetLog* net_log,
bool secure_transport_restricted,
- const std::vector<net::ProxyServer>& proxies_for_http,
- const std::vector<net::ProxyServer>& proxies_for_https) {
+ const std::vector<net::ProxyServer>& proxies_for_http) {
DCHECK(thread_checker_.CalledOnValidThread());
const net::NetLog::ParametersCallback& parameters_callback =
base::Bind(&EnableDataReductionProxyCallback, secure_transport_restricted,
- proxies_for_http, proxies_for_https);
+ proxies_for_http);
PostEnabledEvent(net_log, net::NetLog::TYPE_DATA_REDUCTION_PROXY_ENABLED,
true, parameters_callback);
}

Powered by Google App Engine
This is Rietveld 408576698