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

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

Issue 2468413009: Revert of Change TWCC send interval to reduce overhead on low BW situations. (Closed)
Patch Set: 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);
51 50
52 static const int kMinSendIntervalMs; 51 static const int kDefaultProcessIntervalMs;
53 static const int kMaxSendIntervalMs;
54 static const int kDefaultSendIntervalMs;
55 static const int kBackWindowMs; 52 static const int kBackWindowMs;
56 53
57 private: 54 private:
58 void OnPacketArrival(uint16_t sequence_number, int64_t arrival_time) 55 void OnPacketArrival(uint16_t sequence_number, int64_t arrival_time)
59 EXCLUSIVE_LOCKS_REQUIRED(&lock_); 56 EXCLUSIVE_LOCKS_REQUIRED(&lock_);
60 bool BuildFeedbackPacket(rtcp::TransportFeedback* feedback_packet); 57 bool BuildFeedbackPacket(rtcp::TransportFeedback* feedback_packet);
61 58
62 Clock* const clock_; 59 Clock* const clock_;
63 PacketRouter* const packet_router_; 60 PacketRouter* const packet_router_;
64 int64_t last_process_time_ms_; 61 int64_t last_process_time_ms_;
65 62
66 rtc::CriticalSection lock_; 63 rtc::CriticalSection lock_;
67 64
68 uint32_t media_ssrc_ GUARDED_BY(&lock_); 65 uint32_t media_ssrc_ GUARDED_BY(&lock_);
69 uint8_t feedback_sequence_ GUARDED_BY(&lock_); 66 uint8_t feedback_sequence_ GUARDED_BY(&lock_);
70 SequenceNumberUnwrapper unwrapper_ GUARDED_BY(&lock_); 67 SequenceNumberUnwrapper unwrapper_ GUARDED_BY(&lock_);
71 int64_t window_start_seq_ GUARDED_BY(&lock_); 68 int64_t window_start_seq_ GUARDED_BY(&lock_);
72 // Map unwrapped seq -> time. 69 // Map unwrapped seq -> time.
73 std::map<int64_t, int64_t> packet_arrival_times_ GUARDED_BY(&lock_); 70 std::map<int64_t, int64_t> packet_arrival_times_ GUARDED_BY(&lock_);
74 int64_t send_interval_ms_ GUARDED_BY(&lock_);
75 }; 71 };
76 72
77 } // namespace webrtc 73 } // namespace webrtc
78 74
79 #endif // WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_REMOTE_ESTIMATOR_PROXY_H_ 75 #endif // WEBRTC_MODULES_REMOTE_BITRATE_ESTIMATOR_REMOTE_ESTIMATOR_PROXY_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698