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

Side by Side Diff: webrtc/video/vie_channel.h

Issue 1669623004: Use CallStats for RTT in Call, rather than VideoSendStream::GetRtt() (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: Cleanup Created 4 years, 10 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) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 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 23 matching lines...) Expand all
34 class ChannelStatsObserver; 34 class ChannelStatsObserver;
35 class Config; 35 class Config;
36 class EncodedImageCallback; 36 class EncodedImageCallback;
37 class I420FrameCallback; 37 class I420FrameCallback;
38 class IncomingVideoStream; 38 class IncomingVideoStream;
39 class PacedSender; 39 class PacedSender;
40 class PacketRouter; 40 class PacketRouter;
41 class PayloadRouter; 41 class PayloadRouter;
42 class ProcessThread; 42 class ProcessThread;
43 class ReceiveStatisticsProxy; 43 class ReceiveStatisticsProxy;
44 class ReportBlockStats; 44 class ReportBlockStats;
åsapersson 2016/02/05 13:50:12 remove
sprang 2016/02/05 14:10:22 Done.
45 class RtcpRttStats; 45 class RtcpRttStats;
46 class ViEChannelProtectionCallback; 46 class ViEChannelProtectionCallback;
47 class ViERTPObserver; 47 class ViERTPObserver;
48 class VideoCodingModule; 48 class VideoCodingModule;
49 class VideoDecoder; 49 class VideoDecoder;
50 class VideoRenderCallback; 50 class VideoRenderCallback;
51 class VoEVideoSync; 51 class VoEVideoSync;
52 52
53 enum StreamType { 53 enum StreamType {
54 kViEStreamTypeNormal = 0, // Normal media stream 54 kViEStreamTypeNormal = 0, // Normal media stream
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
141 141
142 // Gets the CName of the incoming stream. 142 // Gets the CName of the incoming stream.
143 int32_t GetRemoteRTCPCName(char rtcp_cname[]); 143 int32_t GetRemoteRTCPCName(char rtcp_cname[]);
144 144
145 // Returns statistics reported by the remote client in an RTCP packet. 145 // Returns statistics reported by the remote client in an RTCP packet.
146 // TODO(pbos): Remove this along with VideoSendStream::GetRtt(). 146 // TODO(pbos): Remove this along with VideoSendStream::GetRtt().
147 int32_t GetSendRtcpStatistics(uint16_t* fraction_lost, 147 int32_t GetSendRtcpStatistics(uint16_t* fraction_lost,
148 uint32_t* cumulative_lost, 148 uint32_t* cumulative_lost,
149 uint32_t* extended_max, 149 uint32_t* extended_max,
150 uint32_t* jitter_samples, 150 uint32_t* jitter_samples,
151 int64_t* rtt_ms); 151 int64_t* rtt_ms);
åsapersson 2016/02/05 13:50:12 remove?
sprang 2016/02/05 14:10:22 Done.
152 152
153 // Called on receipt of RTCP report block from remote side. 153 // Called on receipt of RTCP report block from remote side.
154 void RegisterSendChannelRtcpStatisticsCallback( 154 void RegisterSendChannelRtcpStatisticsCallback(
155 RtcpStatisticsCallback* callback); 155 RtcpStatisticsCallback* callback);
156 156
157 // Called on generation of RTCP stats 157 // Called on generation of RTCP stats
158 void RegisterReceiveChannelRtcpStatisticsCallback( 158 void RegisterReceiveChannelRtcpStatisticsCallback(
159 RtcpStatisticsCallback* callback); 159 RtcpStatisticsCallback* callback);
160 160
161 // Gets send statistics for the rtp and rtx stream. 161 // Gets send statistics for the rtp and rtx stream.
(...skipping 265 matching lines...) Expand 10 before | Expand all | Expand 10 after
427 427
428 const rtc::scoped_ptr<RtcpBandwidthObserver> bandwidth_observer_; 428 const rtc::scoped_ptr<RtcpBandwidthObserver> bandwidth_observer_;
429 TransportFeedbackObserver* const transport_feedback_observer_; 429 TransportFeedbackObserver* const transport_feedback_observer_;
430 430
431 rtc::PlatformThread decode_thread_; 431 rtc::PlatformThread decode_thread_;
432 432
433 int nack_history_size_sender_; 433 int nack_history_size_sender_;
434 int max_nack_reordering_threshold_; 434 int max_nack_reordering_threshold_;
435 I420FrameCallback* pre_render_callback_ GUARDED_BY(crit_); 435 I420FrameCallback* pre_render_callback_ GUARDED_BY(crit_);
436 436
437 const rtc::scoped_ptr<ReportBlockStats> report_block_stats_sender_;
438
439 int64_t time_of_first_rtt_ms_ GUARDED_BY(crit_); 437 int64_t time_of_first_rtt_ms_ GUARDED_BY(crit_);
440 int64_t rtt_sum_ms_ GUARDED_BY(crit_); 438 int64_t rtt_sum_ms_ GUARDED_BY(crit_);
441 int64_t last_rtt_ms_ GUARDED_BY(crit_); 439 int64_t last_rtt_ms_ GUARDED_BY(crit_);
442 size_t num_rtts_ GUARDED_BY(crit_); 440 size_t num_rtts_ GUARDED_BY(crit_);
443 441
444 // RtpRtcp modules, declared last as they use other members on construction. 442 // RtpRtcp modules, declared last as they use other members on construction.
445 const std::vector<RtpRtcp*> rtp_rtcp_modules_; 443 const std::vector<RtpRtcp*> rtp_rtcp_modules_;
446 size_t num_active_rtp_rtcp_modules_ GUARDED_BY(crit_); 444 size_t num_active_rtp_rtcp_modules_ GUARDED_BY(crit_);
447 }; 445 };
448 446
449 } // namespace webrtc 447 } // namespace webrtc
450 448
451 #endif // WEBRTC_VIDEO_VIE_CHANNEL_H_ 449 #endif // WEBRTC_VIDEO_VIE_CHANNEL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698