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

Side by Side Diff: webrtc/modules/remote_bitrate_estimator/remote_estimator_proxy.h

Issue 2381833003: Change TWCC send interval to reduce overhead on low BW situations. (Closed)
Patch Set: fix nit's 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
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 29 matching lines...) Expand all
40 void IncomingPacket(int64_t arrival_time_ms, 40 void IncomingPacket(int64_t arrival_time_ms,
41 size_t payload_size, 41 size_t payload_size,
42 const RTPHeader& header) override; 42 const RTPHeader& header) override;
43 void RemoveStream(uint32_t ssrc) override {} 43 void RemoveStream(uint32_t ssrc) override {}
44 bool LatestEstimate(std::vector<unsigned int>* ssrcs, 44 bool LatestEstimate(std::vector<unsigned int>* ssrcs,
45 unsigned int* bitrate_bps) const override; 45 unsigned int* bitrate_bps) const override;
46 void OnRttUpdate(int64_t avg_rtt_ms, int64_t max_rtt_ms) override {} 46 void OnRttUpdate(int64_t avg_rtt_ms, int64_t max_rtt_ms) override {}
47 void SetMinBitrate(int min_bitrate_bps) override {} 47 void SetMinBitrate(int min_bitrate_bps) override {}
48 int64_t TimeUntilNextProcess() override; 48 int64_t TimeUntilNextProcess() override;
49 void Process() override; 49 void Process() override;
50 void OnBitrateChanged(int bitrate);
50 51
51 static const int kDefaultProcessIntervalMs; 52 static const int kMinSendIntervalMs;
53 static const int kMaxSendIntervalMs;
54 static const int kDefaultSendIntervalMs;
52 static const int kBackWindowMs; 55 static const int kBackWindowMs;
53 56
54 private: 57 private:
55 void OnPacketArrival(uint16_t sequence_number, int64_t arrival_time) 58 void OnPacketArrival(uint16_t sequence_number, int64_t arrival_time)
56 EXCLUSIVE_LOCKS_REQUIRED(&lock_); 59 EXCLUSIVE_LOCKS_REQUIRED(&lock_);
57 bool BuildFeedbackPacket(rtcp::TransportFeedback* feedback_packet); 60 bool BuildFeedbackPacket(rtcp::TransportFeedback* feedback_packet);
58 61
59 Clock* const clock_; 62 Clock* const clock_;
60 PacketRouter* const packet_router_; 63 PacketRouter* const packet_router_;
61 int64_t last_process_time_ms_; 64 int64_t last_process_time_ms_;
62 65
63 rtc::CriticalSection lock_; 66 rtc::CriticalSection lock_;
64 67
65 uint32_t media_ssrc_ GUARDED_BY(&lock_); 68 uint32_t media_ssrc_ GUARDED_BY(&lock_);
66 uint8_t feedback_sequence_ GUARDED_BY(&lock_); 69 uint8_t feedback_sequence_ GUARDED_BY(&lock_);
67 SequenceNumberUnwrapper unwrapper_ GUARDED_BY(&lock_); 70 SequenceNumberUnwrapper unwrapper_ GUARDED_BY(&lock_);
68 int64_t window_start_seq_ GUARDED_BY(&lock_); 71 int64_t window_start_seq_ GUARDED_BY(&lock_);
69 // Map unwrapped seq -> time. 72 // Map unwrapped seq -> time.
70 std::map<int64_t, int64_t> packet_arrival_times_ GUARDED_BY(&lock_); 73 std::map<int64_t, int64_t> packet_arrival_times_ GUARDED_BY(&lock_);
74 int64_t send_interval_ms_ GUARDED_BY(&lock_);
71 }; 75 };
72 76
73 } // namespace webrtc 77 } // namespace webrtc
74 78
75 #endif // WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_REMOTE_ESTIMATOR_PROXY_H_ 79 #endif // WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_REMOTE_ESTIMATOR_PROXY_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698