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

Side by Side Diff: webrtc/audio/audio_receive_stream.cc

Issue 2503383002: Expose RtpCodecParameters to VoiceMediaInfo stats. (Closed)
Patch Set: Addressed comments, using int Created 4 years, 1 month 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/api/call/audio_send_stream.cc ('k') | webrtc/audio/audio_send_stream.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 (c) 2015 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2015 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 169 matching lines...) Expand 10 before | Expand all | Expand 10 after
180 return stats; 180 return stats;
181 } 181 }
182 182
183 stats.bytes_rcvd = call_stats.bytesReceived; 183 stats.bytes_rcvd = call_stats.bytesReceived;
184 stats.packets_rcvd = call_stats.packetsReceived; 184 stats.packets_rcvd = call_stats.packetsReceived;
185 stats.packets_lost = call_stats.cumulativeLost; 185 stats.packets_lost = call_stats.cumulativeLost;
186 stats.fraction_lost = Q8ToFloat(call_stats.fractionLost); 186 stats.fraction_lost = Q8ToFloat(call_stats.fractionLost);
187 stats.capture_start_ntp_time_ms = call_stats.capture_start_ntp_time_ms_; 187 stats.capture_start_ntp_time_ms = call_stats.capture_start_ntp_time_ms_;
188 if (codec_inst.pltype != -1) { 188 if (codec_inst.pltype != -1) {
189 stats.codec_name = codec_inst.plname; 189 stats.codec_name = codec_inst.plname;
190 stats.codec_payload_type = rtc::Optional<int>(codec_inst.pltype);
190 } 191 }
191 stats.ext_seqnum = call_stats.extendedMax; 192 stats.ext_seqnum = call_stats.extendedMax;
192 if (codec_inst.plfreq / 1000 > 0) { 193 if (codec_inst.plfreq / 1000 > 0) {
193 stats.jitter_ms = call_stats.jitterSamples / (codec_inst.plfreq / 1000); 194 stats.jitter_ms = call_stats.jitterSamples / (codec_inst.plfreq / 1000);
194 } 195 }
195 stats.delay_estimate_ms = channel_proxy_->GetDelayEstimate(); 196 stats.delay_estimate_ms = channel_proxy_->GetDelayEstimate();
196 stats.audio_level = channel_proxy_->GetSpeechOutputLevelFullRange(); 197 stats.audio_level = channel_proxy_->GetSpeechOutputLevelFullRange();
197 198
198 // Get jitter buffer and total delay (alg + jitter + playout) stats. 199 // Get jitter buffer and total delay (alg + jitter + playout) stats.
199 auto ns = channel_proxy_->GetNetworkStatistics(); 200 auto ns = channel_proxy_->GetNetworkStatistics();
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
300 301
301 VoiceEngine* AudioReceiveStream::voice_engine() const { 302 VoiceEngine* AudioReceiveStream::voice_engine() const {
302 internal::AudioState* audio_state = 303 internal::AudioState* audio_state =
303 static_cast<internal::AudioState*>(audio_state_.get()); 304 static_cast<internal::AudioState*>(audio_state_.get());
304 VoiceEngine* voice_engine = audio_state->voice_engine(); 305 VoiceEngine* voice_engine = audio_state->voice_engine();
305 RTC_DCHECK(voice_engine); 306 RTC_DCHECK(voice_engine);
306 return voice_engine; 307 return voice_engine;
307 } 308 }
308 } // namespace internal 309 } // namespace internal
309 } // namespace webrtc 310 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/api/call/audio_send_stream.cc ('k') | webrtc/audio/audio_send_stream.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698