Commit 0002c914 authored by Yuwei Huang's avatar Yuwei Huang Committed by Commit Bot

[remoting][host] Enable TCP by default

The host TCP experiment has been rolled out to 100% and we are not
seeing any regressions, so this CL makes TCP enabled on the host by
default. It can still be disabled by adding a `Disable-TCP` host
experiment.

Bug: 600032
Change-Id: Ic5694049965e510a69c7c4c547d8d9567c05d997
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2350390
Auto-Submit: Yuwei Huang <yuweih@chromium.org>
Commit-Queue: Joe Downing <joedow@chromium.org>
Reviewed-by: default avatarJoe Downing <joedow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#797132}
parent 355e6777
......@@ -443,14 +443,13 @@ rtc::AsyncPacketSocket* ChromiumPacketSocketFactory::CreateClientTcpSocket(
const rtc::ProxyInfo& proxy_info,
const std::string& user_agent,
const rtc::PacketSocketTcpOptions& opts) {
if (!session_options_provider_ ||
!session_options_provider_->session_options().GetBoolValue(
"Enable-TCP")) {
HOST_LOG << "Enable-TCP experiment is not enabled. Client TCP socket won't "
<< "be created.";
if (session_options_provider_ &&
session_options_provider_->session_options().GetBoolValue(
"Disable-TCP")) {
HOST_LOG << "Disable-TCP experiment is enabled. Client TCP socket won't be "
<< "created.";
return nullptr;
}
HOST_LOG << "Enable-TCP experiment is enabled. Creating client TCP socket...";
auto socket = std::make_unique<StreamPacketSocket>();
if (!socket->InitClientTcp(local_address, remote_address, proxy_info,
user_agent, opts)) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment