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

Side by Side Diff: webrtc/modules/pacing/paced_sender.h

Issue 2970653004: Reimplemeted "Test and fix for huge bwe drop after alr state" (Closed)
Patch Set: Fix uninitialized variable Created 3 years, 5 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 109 matching lines...) Expand 10 before | Expand all | Expand 10 after
120 virtual size_t QueueSizePackets() const; 120 virtual size_t QueueSizePackets() const;
121 121
122 // Returns the time when the first packet was sent, or -1 if no packet is 122 // Returns the time when the first packet was sent, or -1 if no packet is
123 // sent. 123 // sent.
124 virtual int64_t FirstSentPacketTimeMs() const; 124 virtual int64_t FirstSentPacketTimeMs() const;
125 125
126 // Returns the number of milliseconds it will take to send the current 126 // Returns the number of milliseconds it will take to send the current
127 // packets in the queue, given the current size and bitrate, ignoring prio. 127 // packets in the queue, given the current size and bitrate, ignoring prio.
128 virtual int64_t ExpectedQueueTimeMs() const; 128 virtual int64_t ExpectedQueueTimeMs() const;
129 129
130 // Returns time in milliseconds when the current application-limited region
131 // started or empty result if the sender is currently not application-limited.
132 //
133 // Application Limited Region (ALR) refers to operating in a state where the
134 // traffic on network is limited due to application not having enough
135 // traffic to meet the current channel capacity.
136 virtual rtc::Optional<int64_t> GetApplicationLimitedRegionStartTime() const; 130 virtual rtc::Optional<int64_t> GetApplicationLimitedRegionStartTime() const;
137 131
138 // Returns the average time since being enqueued, in milliseconds, for all 132 // Returns the average time since being enqueued, in milliseconds, for all
139 // packets currently in the pacer queue, or 0 if queue is empty. 133 // packets currently in the pacer queue, or 0 if queue is empty.
140 virtual int64_t AverageQueueTimeMs(); 134 virtual int64_t AverageQueueTimeMs();
141 135
142 // Returns the number of milliseconds until the module want a worker thread 136 // Returns the number of milliseconds until the module want a worker thread
143 // to call Process. 137 // to call Process.
144 int64_t TimeUntilNextProcess() override; 138 int64_t TimeUntilNextProcess() override;
145 139
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
193 187
194 std::unique_ptr<paced_sender::PacketQueue> packets_ GUARDED_BY(critsect_); 188 std::unique_ptr<paced_sender::PacketQueue> packets_ GUARDED_BY(critsect_);
195 uint64_t packet_counter_; 189 uint64_t packet_counter_;
196 ProcessThread* process_thread_ = nullptr; 190 ProcessThread* process_thread_ = nullptr;
197 191
198 float pacing_factor_ GUARDED_BY(critsect_); 192 float pacing_factor_ GUARDED_BY(critsect_);
199 int64_t queue_time_limit GUARDED_BY(critsect_); 193 int64_t queue_time_limit GUARDED_BY(critsect_);
200 }; 194 };
201 } // namespace webrtc 195 } // namespace webrtc
202 #endif // WEBRTC_MODULES_PACING_PACED_SENDER_H_ 196 #endif // WEBRTC_MODULES_PACING_PACED_SENDER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698