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

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

Issue 1478253002: Add histogram stats for send delay for a sent video stream. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Created 4 years, 8 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) 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 279 matching lines...) Expand 10 before | Expand all | Expand 10 after
290 RTC_DCHECK_GE(config.encoder_settings.payload_type, 0); 290 RTC_DCHECK_GE(config.encoder_settings.payload_type, 0);
291 RTC_DCHECK_LE(config.encoder_settings.payload_type, 127); 291 RTC_DCHECK_LE(config.encoder_settings.payload_type, 127);
292 RTC_CHECK_EQ(0, vie_encoder_.RegisterExternalEncoder( 292 RTC_CHECK_EQ(0, vie_encoder_.RegisterExternalEncoder(
293 config.encoder_settings.encoder, 293 config.encoder_settings.encoder,
294 config.encoder_settings.payload_type, 294 config.encoder_settings.payload_type,
295 config.encoder_settings.internal_source)); 295 config.encoder_settings.internal_source));
296 296
297 ReconfigureVideoEncoder(encoder_config); 297 ReconfigureVideoEncoder(encoder_config);
298 298
299 vie_channel_.RegisterSendSideDelayObserver(&stats_proxy_); 299 vie_channel_.RegisterSendSideDelayObserver(&stats_proxy_);
300 vie_channel_.RegisterSendPacketObserver(&stats_proxy_);
300 301
301 if (config_.post_encode_callback) 302 if (config_.post_encode_callback)
302 vie_encoder_.RegisterPostEncodeImageCallback(&encoded_frame_proxy_); 303 vie_encoder_.RegisterPostEncodeImageCallback(&encoded_frame_proxy_);
303 304
304 if (config_.suspend_below_min_bitrate) { 305 if (config_.suspend_below_min_bitrate) {
305 vcm_->SuspendBelowMinBitrate(); 306 vcm_->SuspendBelowMinBitrate();
306 bitrate_allocator_->EnforceMinBitrate(false); 307 bitrate_allocator_->EnforceMinBitrate(false);
307 } 308 }
308 309
309 vie_channel_.RegisterRtcpPacketTypeCounterObserver(&stats_proxy_); 310 vie_channel_.RegisterRtcpPacketTypeCounterObserver(&stats_proxy_);
(...skipping 318 matching lines...) Expand 10 before | Expand all | Expand 10 after
628 int VideoSendStream::GetPaddingNeededBps() const { 629 int VideoSendStream::GetPaddingNeededBps() const {
629 return vie_encoder_.GetPaddingNeededBps(); 630 return vie_encoder_.GetPaddingNeededBps();
630 } 631 }
631 632
632 void VideoSendStream::OnBitrateUpdated(uint32_t bitrate_bps, 633 void VideoSendStream::OnBitrateUpdated(uint32_t bitrate_bps,
633 uint8_t fraction_loss, 634 uint8_t fraction_loss,
634 int64_t rtt) { 635 int64_t rtt) {
635 vie_encoder_.OnBitrateUpdated(bitrate_bps, fraction_loss, rtt); 636 vie_encoder_.OnBitrateUpdated(bitrate_bps, fraction_loss, rtt);
636 } 637 }
637 638
639 bool VideoSendStream::OnSentPacket(int packet_id) {
640 return stats_proxy_.OnSentPacket(packet_id);
641 }
642
638 } // namespace internal 643 } // namespace internal
639 } // namespace webrtc 644 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698