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

Side by Side Diff: webrtc/voice_engine/test/auto_test/standard/codec_test.cc

Issue 1316523002: Convert channel counts to size_t. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc@master
Patch Set: Fix compile Created 4 years, 11 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 | « webrtc/voice_engine/output_mixer.cc ('k') | webrtc/voice_engine/test/cmd_test/voe_cmd_test.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) 2011 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2011 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 32 matching lines...) Expand 10 before | Expand all | Expand 10 after
43 } 43 }
44 44
45 static bool IsNotViableSendCodec(const char* codec_name) { 45 static bool IsNotViableSendCodec(const char* codec_name) {
46 return !_stricmp(codec_name, "CN") || 46 return !_stricmp(codec_name, "CN") ||
47 !_stricmp(codec_name, "telephone-event") || 47 !_stricmp(codec_name, "telephone-event") ||
48 !_stricmp(codec_name, "red"); 48 !_stricmp(codec_name, "red");
49 } 49 }
50 50
51 TEST_F(CodecTest, PcmuIsDefaultCodecAndHasTheRightValues) { 51 TEST_F(CodecTest, PcmuIsDefaultCodecAndHasTheRightValues) {
52 EXPECT_EQ(0, voe_codec_->GetSendCodec(channel_, codec_instance_)); 52 EXPECT_EQ(0, voe_codec_->GetSendCodec(channel_, codec_instance_));
53 EXPECT_EQ(1, codec_instance_.channels); 53 EXPECT_EQ(1u, codec_instance_.channels);
54 EXPECT_EQ(160, codec_instance_.pacsize); 54 EXPECT_EQ(160, codec_instance_.pacsize);
55 EXPECT_EQ(8000, codec_instance_.plfreq); 55 EXPECT_EQ(8000, codec_instance_.plfreq);
56 EXPECT_EQ(0, codec_instance_.pltype); 56 EXPECT_EQ(0, codec_instance_.pltype);
57 EXPECT_EQ(64000, codec_instance_.rate); 57 EXPECT_EQ(64000, codec_instance_.rate);
58 EXPECT_STRCASEEQ("PCMU", codec_instance_.plname); 58 EXPECT_STRCASEEQ("PCMU", codec_instance_.plname);
59 } 59 }
60 60
61 TEST_F(CodecTest, VoiceActivityDetectionIsOffByDefault) { 61 TEST_F(CodecTest, VoiceActivityDetectionIsOffByDefault) {
62 bool vad_enabled = false; 62 bool vad_enabled = false;
63 bool dtx_disabled = false; 63 bool dtx_disabled = false;
(...skipping 166 matching lines...) Expand 10 before | Expand all | Expand 10 after
230 TEST_LOG("%d ", packet_size); 230 TEST_LOG("%d ", packet_size);
231 TEST_LOG_FLUSH; 231 TEST_LOG_FLUSH;
232 at_least_one_succeeded = true; 232 at_least_one_succeeded = true;
233 Sleep(CODEC_TEST_TIME); 233 Sleep(CODEC_TEST_TIME);
234 } 234 }
235 } 235 }
236 TEST_LOG("\n"); 236 TEST_LOG("\n");
237 EXPECT_TRUE(at_least_one_succeeded); 237 EXPECT_TRUE(at_least_one_succeeded);
238 } 238 }
239 } 239 }
OLDNEW
« no previous file with comments | « webrtc/voice_engine/output_mixer.cc ('k') | webrtc/voice_engine/test/cmd_test/voe_cmd_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698