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

Side by Side Diff: webrtc/modules/bitrate_controller/send_side_bandwidth_estimation.h

Issue 2306963002: Only log BWE updates if the actual estimate changed or if we have non-zero loss reports. (Closed) Base URL: https://chromium.googlesource.com/external/webrtc.git@master
Patch Set: . Created 4 years, 3 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 | « no previous file | webrtc/modules/bitrate_controller/send_side_bandwidth_estimation.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) 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 * FEC and NACK added bitrate is handled outside class 10 * FEC and NACK added bitrate is handled outside class
(...skipping 66 matching lines...) Expand 10 before | Expand all | Expand 10 after
77 int expected_packets_since_last_loss_update_; 77 int expected_packets_since_last_loss_update_;
78 78
79 uint32_t bitrate_; 79 uint32_t bitrate_;
80 uint32_t min_bitrate_configured_; 80 uint32_t min_bitrate_configured_;
81 uint32_t max_bitrate_configured_; 81 uint32_t max_bitrate_configured_;
82 int64_t last_low_bitrate_log_ms_; 82 int64_t last_low_bitrate_log_ms_;
83 83
84 bool has_decreased_since_last_fraction_loss_; 84 bool has_decreased_since_last_fraction_loss_;
85 int64_t time_last_receiver_block_ms_; 85 int64_t time_last_receiver_block_ms_;
86 uint8_t last_fraction_loss_; 86 uint8_t last_fraction_loss_;
87 uint8_t last_logged_fraction_loss_;
87 int64_t last_round_trip_time_ms_; 88 int64_t last_round_trip_time_ms_;
88 89
89 uint32_t bwe_incoming_; 90 uint32_t bwe_incoming_;
90 uint32_t delay_based_bitrate_bps_; 91 uint32_t delay_based_bitrate_bps_;
91 int64_t time_last_decrease_ms_; 92 int64_t time_last_decrease_ms_;
92 int64_t first_report_time_ms_; 93 int64_t first_report_time_ms_;
93 int initially_lost_packets_; 94 int initially_lost_packets_;
94 int bitrate_at_2_seconds_kbps_; 95 int bitrate_at_2_seconds_kbps_;
95 UmaState uma_update_state_; 96 UmaState uma_update_state_;
96 std::vector<bool> rampup_uma_stats_updated_; 97 std::vector<bool> rampup_uma_stats_updated_;
97 RtcEventLog* event_log_; 98 RtcEventLog* event_log_;
99 int64_t last_rtc_event_log_ms_;
98 }; 100 };
99 } // namespace webrtc 101 } // namespace webrtc
100 #endif // WEBRTC_MODULES_BITRATE_CONTROLLER_SEND_SIDE_BANDWIDTH_ESTIMATION_H_ 102 #endif // WEBRTC_MODULES_BITRATE_CONTROLLER_SEND_SIDE_BANDWIDTH_ESTIMATION_H_
OLDNEW
« no previous file with comments | « no previous file | webrtc/modules/bitrate_controller/send_side_bandwidth_estimation.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698