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

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

Issue 2735423002: Mark |Clock*| as |const Clock*| (for some CongestionController and BWE related modules) (Closed)
Patch Set: Rebased Created 3 years, 9 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) 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 15 matching lines...) Expand all
26 const int RemoteEstimatorProxy::kBackWindowMs = 500; 26 const int RemoteEstimatorProxy::kBackWindowMs = 500;
27 const int RemoteEstimatorProxy::kMinSendIntervalMs = 50; 27 const int RemoteEstimatorProxy::kMinSendIntervalMs = 50;
28 const int RemoteEstimatorProxy::kMaxSendIntervalMs = 250; 28 const int RemoteEstimatorProxy::kMaxSendIntervalMs = 250;
29 const int RemoteEstimatorProxy::kDefaultSendIntervalMs = 100; 29 const int RemoteEstimatorProxy::kDefaultSendIntervalMs = 100;
30 30
31 // The maximum allowed value for a timestamp in milliseconds. This is lower 31 // The maximum allowed value for a timestamp in milliseconds. This is lower
32 // than the numerical limit since we often convert to microseconds. 32 // than the numerical limit since we often convert to microseconds.
33 static constexpr int64_t kMaxTimeMs = 33 static constexpr int64_t kMaxTimeMs =
34 std::numeric_limits<int64_t>::max() / 1000; 34 std::numeric_limits<int64_t>::max() / 1000;
35 35
36 RemoteEstimatorProxy::RemoteEstimatorProxy(Clock* clock, 36 RemoteEstimatorProxy::RemoteEstimatorProxy(const Clock* clock,
37 PacketRouter* packet_router) 37 PacketRouter* packet_router)
38 : clock_(clock), 38 : clock_(clock),
39 packet_router_(packet_router), 39 packet_router_(packet_router),
40 last_process_time_ms_(-1), 40 last_process_time_ms_(-1),
41 media_ssrc_(0), 41 media_ssrc_(0),
42 feedback_sequence_(0), 42 feedback_sequence_(0),
43 window_start_seq_(-1), 43 window_start_seq_(-1),
44 send_interval_ms_(kDefaultSendIntervalMs) {} 44 send_interval_ms_(kDefaultSendIntervalMs) {}
45 45
46 RemoteEstimatorProxy::~RemoteEstimatorProxy() {} 46 RemoteEstimatorProxy::~RemoteEstimatorProxy() {}
(...skipping 143 matching lines...) Expand 10 before | Expand all | Expand 10 after
190 // Note: Don't erase items from packet_arrival_times_ after sending, in case 190 // Note: Don't erase items from packet_arrival_times_ after sending, in case
191 // they need to be re-sent after a reordering. Removal will be handled 191 // they need to be re-sent after a reordering. Removal will be handled
192 // by OnPacketArrival once packets are too old. 192 // by OnPacketArrival once packets are too old.
193 window_start_seq_ = it->first + 1; 193 window_start_seq_ = it->first + 1;
194 } 194 }
195 195
196 return true; 196 return true;
197 } 197 }
198 198
199 } // namespace webrtc 199 } // namespace webrtc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698