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

Side by Side Diff: webrtc/test/call_test.h

Issue 2361803003: Add logging statements to places where the frame might be dropped in WebRTC pipeline. (Closed)
Patch Set: Remove the DCHECK since it will crash anyway. Created 4 years, 2 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) 2014 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2014 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 #ifndef WEBRTC_TEST_CALL_TEST_H_ 10 #ifndef WEBRTC_TEST_CALL_TEST_H_
11 #define WEBRTC_TEST_CALL_TEST_H_ 11 #define WEBRTC_TEST_CALL_TEST_H_
12 12
13 #include <memory> 13 #include <memory>
14 #include <vector> 14 #include <vector>
15 15
16 #include "webrtc/call.h" 16 #include "webrtc/call.h"
17 #include "webrtc/test/fake_audio_device.h" 17 #include "webrtc/test/fake_audio_device.h"
18 #include "webrtc/test/fake_decoder.h" 18 #include "webrtc/test/fake_decoder.h"
19 #include "webrtc/test/fake_encoder.h" 19 #include "webrtc/test/fake_encoder.h"
20 #include "webrtc/test/fake_videorenderer.h"
20 #include "webrtc/test/frame_generator_capturer.h" 21 #include "webrtc/test/frame_generator_capturer.h"
21 #include "webrtc/test/rtp_rtcp_observer.h" 22 #include "webrtc/test/rtp_rtcp_observer.h"
22 23
23 namespace webrtc { 24 namespace webrtc {
24 25
25 class VoEBase; 26 class VoEBase;
26 class VoECodec; 27 class VoECodec;
27 28
28 namespace test { 29 namespace test {
29 30
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
96 std::vector<VideoReceiveStream*> video_receive_streams_; 97 std::vector<VideoReceiveStream*> video_receive_streams_;
97 std::vector<AudioReceiveStream::Config> audio_receive_configs_; 98 std::vector<AudioReceiveStream::Config> audio_receive_configs_;
98 std::vector<AudioReceiveStream*> audio_receive_streams_; 99 std::vector<AudioReceiveStream*> audio_receive_streams_;
99 100
100 std::unique_ptr<test::FrameGeneratorCapturer> frame_generator_capturer_; 101 std::unique_ptr<test::FrameGeneratorCapturer> frame_generator_capturer_;
101 test::FakeEncoder fake_encoder_; 102 test::FakeEncoder fake_encoder_;
102 std::vector<std::unique_ptr<VideoDecoder>> allocated_decoders_; 103 std::vector<std::unique_ptr<VideoDecoder>> allocated_decoders_;
103 size_t num_video_streams_; 104 size_t num_video_streams_;
104 size_t num_audio_streams_; 105 size_t num_audio_streams_;
105 rtc::scoped_refptr<AudioDecoderFactory> decoder_factory_; 106 rtc::scoped_refptr<AudioDecoderFactory> decoder_factory_;
107 test::FakeVideoRenderer fake_renderer_;
106 108
107 private: 109 private:
108 // TODO(holmer): Remove once VoiceEngine is fully refactored to the new API. 110 // TODO(holmer): Remove once VoiceEngine is fully refactored to the new API.
109 // These methods are used to set up legacy voice engines and channels which is 111 // These methods are used to set up legacy voice engines and channels which is
110 // necessary while voice engine is being refactored to the new stream API. 112 // necessary while voice engine is being refactored to the new stream API.
111 struct VoiceEngineState { 113 struct VoiceEngineState {
112 VoiceEngineState() 114 VoiceEngineState()
113 : voice_engine(nullptr), 115 : voice_engine(nullptr),
114 base(nullptr), 116 base(nullptr),
115 codec(nullptr), 117 codec(nullptr),
(...skipping 64 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 public: 182 public:
181 explicit EndToEndTest(unsigned int timeout_ms); 183 explicit EndToEndTest(unsigned int timeout_ms);
182 184
183 bool ShouldCreateReceivers() const override; 185 bool ShouldCreateReceivers() const override;
184 }; 186 };
185 187
186 } // namespace test 188 } // namespace test
187 } // namespace webrtc 189 } // namespace webrtc
188 190
189 #endif // WEBRTC_TEST_CALL_TEST_H_ 191 #endif // WEBRTC_TEST_CALL_TEST_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698