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

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

Issue 2042603002: Movable support for VideoReceiveStream::Config and avoid copies (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Add TODO Created 4 years, 6 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/video/video_receive_stream.h ('k') | webrtc/video_receive_stream.h » ('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) 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
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
140 Call::Config::kDefaultStartBitrateBps / 1000; 140 Call::Config::kDefaultStartBitrateBps / 1000;
141 141
142 return codec; 142 return codec;
143 } 143 }
144 } // namespace 144 } // namespace
145 145
146 namespace internal { 146 namespace internal {
147 VideoReceiveStream::VideoReceiveStream( 147 VideoReceiveStream::VideoReceiveStream(
148 int num_cpu_cores, 148 int num_cpu_cores,
149 CongestionController* congestion_controller, 149 CongestionController* congestion_controller,
150 const VideoReceiveStream::Config& config, 150 VideoReceiveStream::Config config,
151 webrtc::VoiceEngine* voice_engine, 151 webrtc::VoiceEngine* voice_engine,
152 ProcessThread* process_thread, 152 ProcessThread* process_thread,
153 CallStats* call_stats, 153 CallStats* call_stats,
154 VieRemb* remb) 154 VieRemb* remb)
155 : transport_adapter_(config.rtcp_send_transport), 155 : transport_adapter_(config.rtcp_send_transport),
156 encoded_frame_proxy_(config.pre_decode_callback), 156 encoded_frame_proxy_(config.pre_decode_callback),
157 config_(config), 157 config_(std::move(config)),
158 process_thread_(process_thread), 158 process_thread_(process_thread),
159 clock_(Clock::GetRealTimeClock()), 159 clock_(Clock::GetRealTimeClock()),
160 decode_thread_(DecodeThreadFunction, this, "DecodingThread"), 160 decode_thread_(DecodeThreadFunction, this, "DecodingThread"),
161 congestion_controller_(congestion_controller), 161 congestion_controller_(congestion_controller),
162 call_stats_(call_stats), 162 call_stats_(call_stats),
163 video_receiver_(clock_, nullptr, this, this, this), 163 video_receiver_(clock_, nullptr, this, this, this),
164 incoming_video_stream_(config.disable_prerenderer_smoothing), 164 incoming_video_stream_(config_.disable_prerenderer_smoothing),
165 stats_proxy_(config_, clock_), 165 stats_proxy_(&config_, clock_),
166 rtp_stream_receiver_(&video_receiver_, 166 rtp_stream_receiver_(&video_receiver_,
167 congestion_controller_->GetRemoteBitrateEstimator( 167 congestion_controller_->GetRemoteBitrateEstimator(
168 UseSendSideBwe(config_)), 168 UseSendSideBwe(config_)),
169 &transport_adapter_, 169 &transport_adapter_,
170 call_stats_->rtcp_rtt_stats(), 170 call_stats_->rtcp_rtt_stats(),
171 congestion_controller_->pacer(), 171 congestion_controller_->pacer(),
172 congestion_controller_->packet_router(), 172 congestion_controller_->packet_router(),
173 remb, 173 remb,
174 config, 174 &config_,
175 &stats_proxy_, 175 &stats_proxy_,
176 process_thread_), 176 process_thread_),
177 video_stream_decoder_(&video_receiver_, 177 video_stream_decoder_(&video_receiver_,
178 &rtp_stream_receiver_, 178 &rtp_stream_receiver_,
179 &rtp_stream_receiver_, 179 &rtp_stream_receiver_,
180 rtp_stream_receiver_.IsRetransmissionsEnabled(), 180 rtp_stream_receiver_.IsRetransmissionsEnabled(),
181 rtp_stream_receiver_.IsFecEnabled(), 181 rtp_stream_receiver_.IsFecEnabled(),
182 &stats_proxy_, 182 &stats_proxy_,
183 &incoming_video_stream_, 183 &incoming_video_stream_,
184 config.pre_render_callback), 184 config.pre_render_callback),
(...skipping 161 matching lines...) Expand 10 before | Expand all | Expand 10 after
346 const std::vector<uint16_t>& sequence_numbers) { 346 const std::vector<uint16_t>& sequence_numbers) {
347 rtp_stream_receiver_.RequestPacketRetransmit(sequence_numbers); 347 rtp_stream_receiver_.RequestPacketRetransmit(sequence_numbers);
348 } 348 }
349 349
350 void VideoReceiveStream::RequestKeyFrame() { 350 void VideoReceiveStream::RequestKeyFrame() {
351 rtp_stream_receiver_.RequestKeyFrame(); 351 rtp_stream_receiver_.RequestKeyFrame();
352 } 352 }
353 353
354 } // namespace internal 354 } // namespace internal
355 } // namespace webrtc 355 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/video/video_receive_stream.h ('k') | webrtc/video_receive_stream.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698