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

Side by Side Diff: webrtc/base/rate_statistics.cc

Issue 2366333003: Fix race / crash in OnNetworkRouteChanged(). (Closed)
Patch Set: Created 4 years, 2 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) 2013 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2013 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
11 #include "webrtc/base/rate_statistics.h" 11 #include "webrtc/base/rate_statistics.h"
12 12
13 #include <algorithm> 13 #include <algorithm>
14 14
15 #include "webrtc/base/checks.h" 15 #include "webrtc/base/checks.h"
16 #include "webrtc/base/logging.h"
16 17
17 namespace webrtc { 18 namespace webrtc {
18 19
19 RateStatistics::RateStatistics(int64_t window_size_ms, float scale) 20 RateStatistics::RateStatistics(int64_t window_size_ms, float scale)
20 : buckets_(new Bucket[window_size_ms]()), 21 : buckets_(new Bucket[window_size_ms]()),
21 accumulated_count_(0), 22 accumulated_count_(0),
22 num_samples_(0), 23 num_samples_(0),
23 oldest_time_(-window_size_ms), 24 oldest_time_(-window_size_ms),
24 oldest_index_(0), 25 oldest_index_(0),
25 scale_(scale), 26 scale_(scale),
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
112 current_window_size_ms_ = window_size_ms; 113 current_window_size_ms_ = window_size_ms;
113 EraseOld(now_ms); 114 EraseOld(now_ms);
114 return true; 115 return true;
115 } 116 }
116 117
117 bool RateStatistics::IsInitialized() const { 118 bool RateStatistics::IsInitialized() const {
118 return oldest_time_ != -max_window_size_ms_; 119 return oldest_time_ != -max_window_size_ms_;
119 } 120 }
120 121
121 } // namespace webrtc 122 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | webrtc/modules/BUILD.gn » ('j') | webrtc/modules/congestion_controller/delay_based_bwe.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698