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

Side by Side Diff: webrtc/modules/audio_coding/test/TestVADDTX.h

Issue 2119393002: Fixed time moving backwards in the AudioCodingModule. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Updated Dtx tests that moved the input timestamp backwards to not do that. Created 4 years, 5 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
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 54 matching lines...) Expand 10 before | Expand all | Expand 10 after
65 // 2 - kAudioFrameCN 65 // 2 - kAudioFrameCN
66 // 3 - kVideoFrameKey (not used by audio) 66 // 3 - kVideoFrameKey (not used by audio)
67 // 4 - kVideoFrameDelta (not used by audio) 67 // 4 - kVideoFrameDelta (not used by audio)
68 void Run(std::string in_filename, int frequency, int channels, 68 void Run(std::string in_filename, int frequency, int channels,
69 std::string out_filename, bool append, const int* expects); 69 std::string out_filename, bool append, const int* expects);
70 70
71 std::unique_ptr<AudioCodingModule> acm_send_; 71 std::unique_ptr<AudioCodingModule> acm_send_;
72 std::unique_ptr<AudioCodingModule> acm_receive_; 72 std::unique_ptr<AudioCodingModule> acm_receive_;
73 std::unique_ptr<Channel> channel_; 73 std::unique_ptr<Channel> channel_;
74 std::unique_ptr<ActivityMonitor> monitor_; 74 std::unique_ptr<ActivityMonitor> monitor_;
75 uint32_t time_stamp_ = 0x12345678;
75 }; 76 };
76 77
77 // TestWebRtcVadDtx is to verify that the WebRTC VAD/DTX perform as they should. 78 // TestWebRtcVadDtx is to verify that the WebRTC VAD/DTX perform as they should.
78 class TestWebRtcVadDtx final : public TestVadDtx { 79 class TestWebRtcVadDtx final : public TestVadDtx {
79 public: 80 public:
80 TestWebRtcVadDtx(); 81 TestWebRtcVadDtx();
81 82
82 void Perform() override; 83 void Perform() override;
83 84
84 private: 85 private:
85 void RunTestCases(); 86 void RunTestCases();
86 void Test(bool new_outfile); 87 void Test(bool new_outfile);
87 void SetVAD(bool enable_dtx, bool enable_vad, ACMVADMode vad_mode); 88 void SetVAD(bool enable_dtx, bool enable_vad, ACMVADMode vad_mode);
88 89
89 bool vad_enabled_; 90 bool vad_enabled_;
90 bool dtx_enabled_; 91 bool dtx_enabled_;
91 int output_file_num_; 92 int output_file_num_;
92 }; 93 };
93 94
94 // TestOpusDtx is to verify that the Opus DTX performs as it should. 95 // TestOpusDtx is to verify that the Opus DTX performs as it should.
95 class TestOpusDtx final : public TestVadDtx { 96 class TestOpusDtx final : public TestVadDtx {
96 public: 97 public:
97 void Perform() override; 98 void Perform() override;
98 }; 99 };
99 100
100 } // namespace webrtc 101 } // namespace webrtc
101 102
102 #endif // WEBRTC_MODULES_AUDIO_CODING_TEST_TESTVADDTX_H_ 103 #endif // WEBRTC_MODULES_AUDIO_CODING_TEST_TESTVADDTX_H_
OLDNEW
« no previous file with comments | « webrtc/modules/audio_coding/acm2/audio_coding_module.cc ('k') | webrtc/modules/audio_coding/test/TestVADDTX.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698