commit 63470298753ad575ae21b00bf9da14c023709617
author Sebastian Jansson <srte@webrtc.org> Fri Feb 01 09:13:43 2019
committer Commit Bot <commit-bot@chromium.org> Fri Feb 01 09:46:59 2019
tree 155351a391b8e9ea7f2407af99679dc44f27a59e
parent f1d1061e51ca8554263323e995fbd534a080f61b

Removes usages of TaskQueueCongestionControl field trial.

It doesn't do anything any more, so it should be removed.

Bug: webrtc:9586
Change-Id: I0b320b6ce4f480ff8cb59451db29bcc77b882b5f
Reviewed-on: https://webrtc-review.googlesource.com/c/120813
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Reviewed-by: Stefan Holmer <stefan@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#26507}

call/rtp_video_sender_unittest.cc

diff --git a/call/rtp_video_sender_unittest.cc b/call/rtp_video_sender_unittest.cc
index 9f98c9d..ede5b1d 100644
--- a/call/rtp_video_sender_unittest.cc
+++ b/call/rtp_video_sender_unittest.cc
@@ -132,23 +132,7 @@
 };
 }  // namespace
 
-class RtpVideoSenderTest : public ::testing::Test,
-                           public ::testing::WithParamInterface<std::string> {
- public:
-  RtpVideoSenderTest() : field_trial_(GetParam()) {}
-
- private:
-  test::ScopedFieldTrials field_trial_;
-};
-
-INSTANTIATE_TEST_SUITE_P(Default, RtpVideoSenderTest, ::testing::Values(""));
-
-INSTANTIATE_TEST_SUITE_P(
-    TaskQueueTrial,
-    RtpVideoSenderTest,
-    ::testing::Values("WebRTC-TaskQueueCongestionControl/Enabled/"));
-
-TEST_P(RtpVideoSenderTest, SendOnOneModule) {
+TEST(RtpVideoSenderTest, SendOnOneModule) {
   uint8_t payload = 'a';
   EncodedImage encoded_image;
   encoded_image.SetTimestamp(1);
@@ -178,7 +162,7 @@
       test.router()->OnEncodedImage(encoded_image, nullptr, nullptr).error);
 }
 
-TEST_P(RtpVideoSenderTest, SendSimulcastSetActive) {
+TEST(RtpVideoSenderTest, SendSimulcastSetActive) {
   uint8_t payload = 'a';
   EncodedImage encoded_image_1;
   encoded_image_1.SetTimestamp(1);
@@ -222,7 +206,7 @@
 // behavior of the payload router. First sets one module to active and checks
 // that outgoing data can be sent on this module, and checks that no data can
 // be sent if both modules are inactive.
-TEST_P(RtpVideoSenderTest, SendSimulcastSetActiveModules) {
+TEST(RtpVideoSenderTest, SendSimulcastSetActiveModules) {
   uint8_t payload = 'a';
   EncodedImage encoded_image_1;
   encoded_image_1.SetTimestamp(1);
@@ -264,7 +248,7 @@
                 .error);
 }
 
-TEST_P(RtpVideoSenderTest, CreateWithNoPreviousStates) {
+TEST(RtpVideoSenderTest, CreateWithNoPreviousStates) {
   RtpVideoSenderTestFixture test({kSsrc1, kSsrc2}, kPayloadType, {});
   test.router()->SetActive(true);
 
@@ -275,7 +259,7 @@
   EXPECT_NE(initial_states.find(kSsrc2), initial_states.end());
 }
 
-TEST_P(RtpVideoSenderTest, CreateWithPreviousStates) {
+TEST(RtpVideoSenderTest, CreateWithPreviousStates) {
   const int64_t kState1SharedFrameId = 123;
   const int64_t kState2SharedFrameId = 234;
   RtpPayloadState state1;
@@ -303,7 +287,7 @@
   EXPECT_EQ(kState2SharedFrameId, initial_states[kSsrc2].shared_frame_id);
 }
 
-TEST_P(RtpVideoSenderTest, FrameCountCallbacks) {
+TEST(RtpVideoSenderTest, FrameCountCallbacks) {
   class MockFrameCountObserver : public FrameCountObserver {
    public:
     MOCK_METHOD2(FrameCountUpdated,