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

Side by Side Diff: webrtc/voice_engine/channel.cc

Issue 2705613002: Rename some variables and methods in RTC event log. (Closed)
Patch Set: Created 3 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
« no previous file with comments | « webrtc/tools/event_log_visualizer/analyzer.cc ('k') | no next file » | 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) 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 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 } 123 }
124 } 124 }
125 125
126 void LogAudioPlayout(uint32_t ssrc) override { 126 void LogAudioPlayout(uint32_t ssrc) override {
127 rtc::CritScope lock(&crit_); 127 rtc::CritScope lock(&crit_);
128 if (event_log_) { 128 if (event_log_) {
129 event_log_->LogAudioPlayout(ssrc); 129 event_log_->LogAudioPlayout(ssrc);
130 } 130 }
131 } 131 }
132 132
133 void LogBwePacketLossEvent(int32_t bitrate, 133 void LogLossBasedBweUpdate(int32_t bitrate_bps,
134 uint8_t fraction_loss, 134 uint8_t fraction_loss,
135 int32_t total_packets) override { 135 int32_t total_packets) override {
136 rtc::CritScope lock(&crit_); 136 rtc::CritScope lock(&crit_);
137 if (event_log_) { 137 if (event_log_) {
138 event_log_->LogBwePacketLossEvent(bitrate, fraction_loss, total_packets); 138 event_log_->LogLossBasedBweUpdate(bitrate_bps, fraction_loss,
139 total_packets);
139 } 140 }
140 } 141 }
141 142
142 void LogBwePacketDelayEvent(int32_t bitrate, 143 void LogDelayBasedBweUpdate(int32_t bitrate_bps,
143 BandwidthUsage detector_state) override { 144 BandwidthUsage detector_state) override {
144 rtc::CritScope lock(&crit_); 145 rtc::CritScope lock(&crit_);
145 if (event_log_) { 146 if (event_log_) {
146 event_log_->LogBwePacketDelayEvent(bitrate, detector_state); 147 event_log_->LogDelayBasedBweUpdate(bitrate_bps, detector_state);
147 } 148 }
148 } 149 }
149 150
150 void LogAudioNetworkAdaptation( 151 void LogAudioNetworkAdaptation(
151 const AudioNetworkAdaptor::EncoderRuntimeConfig& config) override { 152 const AudioNetworkAdaptor::EncoderRuntimeConfig& config) override {
152 rtc::CritScope lock(&crit_); 153 rtc::CritScope lock(&crit_);
153 if (event_log_) { 154 if (event_log_) {
154 event_log_->LogAudioNetworkAdaptation(config); 155 event_log_->LogAudioNetworkAdaptation(config);
155 } 156 }
156 } 157 }
(...skipping 2862 matching lines...) Expand 10 before | Expand all | Expand 10 after
3019 int64_t min_rtt = 0; 3020 int64_t min_rtt = 0;
3020 if (_rtpRtcpModule->RTT(remoteSSRC, &rtt, &avg_rtt, &min_rtt, &max_rtt) != 3021 if (_rtpRtcpModule->RTT(remoteSSRC, &rtt, &avg_rtt, &min_rtt, &max_rtt) !=
3021 0) { 3022 0) {
3022 return 0; 3023 return 0;
3023 } 3024 }
3024 return rtt; 3025 return rtt;
3025 } 3026 }
3026 3027
3027 } // namespace voe 3028 } // namespace voe
3028 } // namespace webrtc 3029 } // namespace webrtc
OLDNEW
« no previous file with comments | « webrtc/tools/event_log_visualizer/analyzer.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698