admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "Enable experiment WebRTC-SendPacketsOnWorkerThread in pc_full_stack_test"

Browse Source 
This reverts commit 1b3f531da404c200da09f229799e827250347b60.

Reason for revert: Simulated network changes has been reverted. 
In order to see the effect of this experiment, there should not be other larger changes affecting the metrics of a few runs. 
https://webrtc.googlesource.com/src/+/baf5c9fabd3eba46a2b7747df00b1124a8f5def8

Original change's description:
> Enable experiment WebRTC-SendPacketsOnWorkerThread in pc_full_stack_test
>
> This is a follow up to https://webrtc-review.googlesource.com/c/src/+/278980 to actually enable the experiment in some tests.
>
> Bug: webrtc:14502
> Change-Id: I166f984bcb94527adc6ebb9169b66abf0f105d76
> Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/279140
> Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
> Commit-Queue: Per Kjellander <perkj@webrtc.org>
> Cr-Commit-Position: refs/heads/main@{#38407}

Bug: webrtc:14502
Change-Id: I6e5a607a284186895d1ecd622fdf28f5c1ffd187
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/279600
Bot-Commit: rubber-stamper@appspot.gserviceaccount.com <rubber-stamper@appspot.gserviceaccount.com>
Commit-Queue: Ilya Nikolaevskiy <ilnik@webrtc.org>
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#38417}
This commit is contained in:
Per Kjellander 2022-10-17 14:00:53 +00:00 committed by WebRTC LUCI CQ
parent 9ea538185a
commit 8b715657fb
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
video/pc_full_stack_tests.cc
View File

@ -97,8 +97,10 @@ std::vector<PCFullStackTestParams> ParameterizedTestParams() {
// Use the network thread as worker thread.
// Use the worker thread for sending packets.
// https://bugs.chromium.org/p/webrtc/issues/detail?id=14502
{.use_network_thread_as_worker_thread = true,
.field_trials = "WebRTC-SendPacketsOnWorkerThread/Enabled/",
{// TODO(webrtc:14502): Enable field trial soon but let it first run a
// couple of times to get a baseline..
// .use_network_thread_as_worker_thread = true,
// .field_trials = "WebRTC-SendPacketsOnWorkerThread/Enabled/",
.test_case_name_postfix = "_ReducedThreads"}};
}