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

Side by Side Diff: webrtc/audio/test/low_bandwidth_audio_test.cc

Issue 2784543002: Revert of Don't hardcode MediaType::ANY in FakeNetworkPipe. (Closed)
Patch Set: Created 3 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 unified diff | Download patch
« no previous file with comments | « no previous file | webrtc/call/bitrate_estimator_tests.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2017 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2017 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 4 * Use of this source code is governed by a BSD-style license
5 * that can be found in the LICENSE file in the root of the source 5 * that can be found in the LICENSE file in the root of the source
6 * tree. An additional intellectual property rights grant can be found 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 62 matching lines...) Expand 10 before | Expand all | Expand 10 after
73 } 73 }
74 74
75 FakeNetworkPipe::Config AudioQualityTest::GetNetworkPipeConfig() { 75 FakeNetworkPipe::Config AudioQualityTest::GetNetworkPipeConfig() {
76 return FakeNetworkPipe::Config(); 76 return FakeNetworkPipe::Config();
77 } 77 }
78 78
79 test::PacketTransport* AudioQualityTest::CreateSendTransport( 79 test::PacketTransport* AudioQualityTest::CreateSendTransport(
80 Call* sender_call) { 80 Call* sender_call) {
81 return new test::PacketTransport( 81 return new test::PacketTransport(
82 sender_call, this, test::PacketTransport::kSender, 82 sender_call, this, test::PacketTransport::kSender,
83 MediaType::AUDIO,
84 GetNetworkPipeConfig()); 83 GetNetworkPipeConfig());
85 } 84 }
86 85
87 test::PacketTransport* AudioQualityTest::CreateReceiveTransport() { 86 test::PacketTransport* AudioQualityTest::CreateReceiveTransport() {
88 return new test::PacketTransport(nullptr, this, 87 return new test::PacketTransport(nullptr, this,
89 test::PacketTransport::kReceiver, MediaType::AUDIO, 88 test::PacketTransport::kReceiver, GetNetworkPipeConfig());
90 GetNetworkPipeConfig());
91 } 89 }
92 90
93 void AudioQualityTest::ModifyAudioConfigs( 91 void AudioQualityTest::ModifyAudioConfigs(
94 AudioSendStream::Config* send_config, 92 AudioSendStream::Config* send_config,
95 std::vector<AudioReceiveStream::Config>* receive_configs) { 93 std::vector<AudioReceiveStream::Config>* receive_configs) {
96 send_config->send_codec_spec.codec_inst = kDefaultCodec; 94 send_config->send_codec_spec.codec_inst = kDefaultCodec;
97 } 95 }
98 96
99 void AudioQualityTest::PerformTest() { 97 void AudioQualityTest::PerformTest() {
100 // Wait until the input audio file is done... 98 // Wait until the input audio file is done...
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
144 } 142 }
145 }; 143 };
146 144
147 TEST_F(LowBandwidthAudioTest, Mobile2GNetwork) { 145 TEST_F(LowBandwidthAudioTest, Mobile2GNetwork) {
148 Mobile2GNetworkTest test; 146 Mobile2GNetworkTest test;
149 RunBaseTest(&test); 147 RunBaseTest(&test);
150 } 148 }
151 149
152 } // namespace test 150 } // namespace test
153 } // namespace webrtc 151 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | webrtc/call/bitrate_estimator_tests.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698