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

Side by Side Diff: webrtc/pc/rtcstatscollector.cc

Issue 2722633005: Move RTCOutboundRTPStreamStats.roundTripTime to inbound, don't collect. (Closed)
Patch Set: Created 3 years, 9 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/pc/rtcstats_integrationtest.cc ('k') | webrtc/pc/rtcstatscollector_unittest.cc » ('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 2016 The WebRTC Project Authors. All rights reserved. 2 * Copyright 2016 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 243 matching lines...) Expand 10 before | Expand all | Expand 10 after
254 const cricket::MediaSenderInfo& media_sender_info, 254 const cricket::MediaSenderInfo& media_sender_info,
255 RTCOutboundRTPStreamStats* outbound_stats) { 255 RTCOutboundRTPStreamStats* outbound_stats) {
256 RTC_DCHECK(outbound_stats); 256 RTC_DCHECK(outbound_stats);
257 outbound_stats->ssrc = media_sender_info.ssrc(); 257 outbound_stats->ssrc = media_sender_info.ssrc();
258 // TODO(hbos): Support the remote case. crbug.com/657856 258 // TODO(hbos): Support the remote case. crbug.com/657856
259 outbound_stats->is_remote = false; 259 outbound_stats->is_remote = false;
260 outbound_stats->packets_sent = 260 outbound_stats->packets_sent =
261 static_cast<uint32_t>(media_sender_info.packets_sent); 261 static_cast<uint32_t>(media_sender_info.packets_sent);
262 outbound_stats->bytes_sent = 262 outbound_stats->bytes_sent =
263 static_cast<uint64_t>(media_sender_info.bytes_sent); 263 static_cast<uint64_t>(media_sender_info.bytes_sent);
264 if (media_sender_info.rtt_ms >= 0) {
265 outbound_stats->round_trip_time = static_cast<double>(
266 media_sender_info.rtt_ms) / rtc::kNumMillisecsPerSec;
267 }
268 } 264 }
269 265
270 void SetOutboundRTPStreamStatsFromVoiceSenderInfo( 266 void SetOutboundRTPStreamStatsFromVoiceSenderInfo(
271 const cricket::VoiceSenderInfo& voice_sender_info, 267 const cricket::VoiceSenderInfo& voice_sender_info,
272 RTCOutboundRTPStreamStats* outbound_audio) { 268 RTCOutboundRTPStreamStats* outbound_audio) {
273 SetOutboundRTPStreamStatsFromMediaSenderInfo( 269 SetOutboundRTPStreamStatsFromMediaSenderInfo(
274 voice_sender_info, outbound_audio); 270 voice_sender_info, outbound_audio);
275 outbound_audio->media_type = "audio"; 271 outbound_audio->media_type = "audio";
276 if (voice_sender_info.codec_payload_type) { 272 if (voice_sender_info.codec_payload_type) {
277 outbound_audio->codec_id = 273 outbound_audio->codec_id =
(...skipping 969 matching lines...) Expand 10 before | Expand all | Expand 10 after
1247 const std::string& type) { 1243 const std::string& type) {
1248 return CandidateTypeToRTCIceCandidateType(type); 1244 return CandidateTypeToRTCIceCandidateType(type);
1249 } 1245 }
1250 1246
1251 const char* DataStateToRTCDataChannelStateForTesting( 1247 const char* DataStateToRTCDataChannelStateForTesting(
1252 DataChannelInterface::DataState state) { 1248 DataChannelInterface::DataState state) {
1253 return DataStateToRTCDataChannelState(state); 1249 return DataStateToRTCDataChannelState(state);
1254 } 1250 }
1255 1251
1256 } // namespace webrtc 1252 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/pc/rtcstats_integrationtest.cc ('k') | webrtc/pc/rtcstatscollector_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698