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

Side by Side Diff: webrtc/video/video_receive_stream.cc

Issue 3008773002: Use RtxReceiveStream. (Closed)
Patch Set: Improve TODO comments. Created 3 years, 3 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
« webrtc/video/BUILD.gn ('K') | « webrtc/video/video_receive_stream.h ('k') | no next file » | 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) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 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
11 #include "webrtc/video/video_receive_stream.h" 11 #include "webrtc/video/video_receive_stream.h"
12 12
13 #include <stdlib.h> 13 #include <stdlib.h>
14 14
15 #include <set> 15 #include <set>
16 #include <string> 16 #include <string>
17 #include <utility> 17 #include <utility>
18 18
19 #include "webrtc/call/rtp_stream_receiver_controller_interface.h" 19 #include "webrtc/call/rtp_stream_receiver_controller_interface.h"
20 #include "webrtc/call/rtx_receive_stream.h"
20 #include "webrtc/common_types.h" 21 #include "webrtc/common_types.h"
21 #include "webrtc/common_video/h264/profile_level_id.h" 22 #include "webrtc/common_video/h264/profile_level_id.h"
22 #include "webrtc/common_video/libyuv/include/webrtc_libyuv.h" 23 #include "webrtc/common_video/libyuv/include/webrtc_libyuv.h"
23 #include "webrtc/modules/rtp_rtcp/include/rtp_receiver.h" 24 #include "webrtc/modules/rtp_rtcp/include/rtp_receiver.h"
24 #include "webrtc/modules/rtp_rtcp/include/rtp_rtcp.h" 25 #include "webrtc/modules/rtp_rtcp/include/rtp_rtcp.h"
25 #include "webrtc/modules/utility/include/process_thread.h" 26 #include "webrtc/modules/utility/include/process_thread.h"
26 #include "webrtc/modules/video_coding/frame_object.h" 27 #include "webrtc/modules/video_coding/frame_object.h"
27 #include "webrtc/modules/video_coding/include/video_coding.h" 28 #include "webrtc/modules/video_coding/include/video_coding.h"
28 #include "webrtc/modules/video_coding/jitter_estimator.h" 29 #include "webrtc/modules/video_coding/jitter_estimator.h"
29 #include "webrtc/modules/video_coding/timing.h" 30 #include "webrtc/modules/video_coding/timing.h"
30 #include "webrtc/modules/video_coding/utility/ivf_file_writer.h" 31 #include "webrtc/modules/video_coding/utility/ivf_file_writer.h"
31 #include "webrtc/rtc_base/checks.h" 32 #include "webrtc/rtc_base/checks.h"
32 #include "webrtc/rtc_base/location.h" 33 #include "webrtc/rtc_base/location.h"
33 #include "webrtc/rtc_base/logging.h" 34 #include "webrtc/rtc_base/logging.h"
34 #include "webrtc/rtc_base/optional.h" 35 #include "webrtc/rtc_base/optional.h"
36 #include "webrtc/rtc_base/ptr_util.h"
35 #include "webrtc/rtc_base/trace_event.h" 37 #include "webrtc/rtc_base/trace_event.h"
36 #include "webrtc/system_wrappers/include/clock.h" 38 #include "webrtc/system_wrappers/include/clock.h"
37 #include "webrtc/system_wrappers/include/field_trial.h" 39 #include "webrtc/system_wrappers/include/field_trial.h"
38 #include "webrtc/video/call_stats.h" 40 #include "webrtc/video/call_stats.h"
39 #include "webrtc/video/receive_statistics_proxy.h" 41 #include "webrtc/video/receive_statistics_proxy.h"
40 42
41 namespace webrtc { 43 namespace webrtc {
42 44
43 namespace { 45 namespace {
44 VideoCodec CreateDecoderVideoCodec(const VideoReceiveStream::Decoder& decoder) { 46 VideoCodec CreateDecoderVideoCodec(const VideoReceiveStream::Decoder& decoder) {
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
113 std::set<int> decoder_payload_types; 115 std::set<int> decoder_payload_types;
114 for (const Decoder& decoder : config_.decoders) { 116 for (const Decoder& decoder : config_.decoders) {
115 RTC_CHECK(decoder.decoder); 117 RTC_CHECK(decoder.decoder);
116 RTC_CHECK(decoder_payload_types.find(decoder.payload_type) == 118 RTC_CHECK(decoder_payload_types.find(decoder.payload_type) ==
117 decoder_payload_types.end()) 119 decoder_payload_types.end())
118 << "Duplicate payload type (" << decoder.payload_type 120 << "Duplicate payload type (" << decoder.payload_type
119 << ") for different decoders."; 121 << ") for different decoders.";
120 decoder_payload_types.insert(decoder.payload_type); 122 decoder_payload_types.insert(decoder.payload_type);
121 } 123 }
122 124
123 video_receiver_.SetRenderDelay(config.render_delay_ms); 125 video_receiver_.SetRenderDelay(config_.render_delay_ms);
124 126
125 jitter_estimator_.reset(new VCMJitterEstimator(clock_)); 127 jitter_estimator_.reset(new VCMJitterEstimator(clock_));
126 frame_buffer_.reset(new video_coding::FrameBuffer( 128 frame_buffer_.reset(new video_coding::FrameBuffer(
127 clock_, jitter_estimator_.get(), timing_.get(), &stats_proxy_)); 129 clock_, jitter_estimator_.get(), timing_.get(), &stats_proxy_));
128 130
129 process_thread_->RegisterModule(&rtp_stream_sync_, RTC_FROM_HERE); 131 process_thread_->RegisterModule(&rtp_stream_sync_, RTC_FROM_HERE);
130 132
131 // Register with RtpStreamReceiverController. 133 // Register with RtpStreamReceiverController.
132 media_receiver_ = receiver_controller->CreateReceiver( 134 media_receiver_ = receiver_controller->CreateReceiver(
133 config_.rtp.remote_ssrc, &rtp_video_stream_receiver_); 135 config_.rtp.remote_ssrc, &rtp_video_stream_receiver_);
134 if (config.rtp.rtx_ssrc) { 136 if (config_.rtp.rtx_ssrc) {
137 rtx_receive_stream_ = rtc::MakeUnique<RtxReceiveStream>(
138 &rtp_video_stream_receiver_, config.rtp.rtx_associated_payload_types,
139 config_.rtp.remote_ssrc);
135 rtx_receiver_ = receiver_controller->CreateReceiver( 140 rtx_receiver_ = receiver_controller->CreateReceiver(
136 config_.rtp.rtx_ssrc, &rtp_video_stream_receiver_); 141 config_.rtp.rtx_ssrc, rtx_receive_stream_.get());
137 } 142 }
138 } 143 }
139 144
140 VideoReceiveStream::~VideoReceiveStream() { 145 VideoReceiveStream::~VideoReceiveStream() {
141 RTC_DCHECK_CALLED_SEQUENTIALLY(&worker_sequence_checker_); 146 RTC_DCHECK_CALLED_SEQUENTIALLY(&worker_sequence_checker_);
142 LOG(LS_INFO) << "~VideoReceiveStream: " << config_.ToString(); 147 LOG(LS_INFO) << "~VideoReceiveStream: " << config_.ToString();
143 Stop(); 148 Stop();
144 149
145 process_thread_->DeRegisterModule(&rtp_stream_sync_); 150 process_thread_->DeRegisterModule(&rtp_stream_sync_);
146 } 151 }
(...skipping 295 matching lines...) Expand 10 before | Expand all | Expand 10 after
442 if (stream_is_active && !receiving_keyframe) { 447 if (stream_is_active && !receiving_keyframe) {
443 LOG(LS_WARNING) << "No decodable frame in " << wait_ms 448 LOG(LS_WARNING) << "No decodable frame in " << wait_ms
444 << " ms, requesting keyframe."; 449 << " ms, requesting keyframe.";
445 RequestKeyFrame(); 450 RequestKeyFrame();
446 } 451 }
447 } 452 }
448 return true; 453 return true;
449 } 454 }
450 } // namespace internal 455 } // namespace internal
451 } // namespace webrtc 456 } // namespace webrtc
OLDNEW
« webrtc/video/BUILD.gn ('K') | « webrtc/video/video_receive_stream.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698