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

Side by Side Diff: media/base/moving_average.cc

Issue 1021943002: Introduce cadence based VideoRendererAlgorithm. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Comments. Created 5 years, 7 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
« no previous file with comments | « media/base/moving_average.h ('k') | media/base/moving_average_unittest.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "media/base/moving_average.h"
6
7 #include <algorithm>
8
9 namespace media {
10
11 MovingAverage::MovingAverage(size_t depth)
12 : depth_(depth), count_(0), samples_(depth_, base::TimeDelta()) {
xhwang 2015/04/28 22:17:39 nit: base::TimeDelta() is not needed as it's the d
DaleCurtis 2015/04/29 00:11:12 Done.
13 }
14
15 MovingAverage::~MovingAverage() {
16 }
17
18 void MovingAverage::AddSample(base::TimeDelta sample) {
19 // |samples_| is zero-initialized, so |oldest| is also zero before |count_|
20 // exceeds |depth_|.
21 base::TimeDelta& oldest = samples_[count_++ % depth_];
22 total_ += sample - oldest;
23 oldest = sample;
24 }
25
26 base::TimeDelta MovingAverage::Average() const {
27 DCHECK_GT(count_, 0u);
28
29 // TODO(dalecurtis): Consider limiting |depth| to powers of two so that we can
30 // replace the integer divide with a bit shift operation.
31
32 return total_ / std::min(depth_, count_);
33 }
34
35 void MovingAverage::Reset() {
36 count_ = 0;
37 total_ = base::TimeDelta();
38 std::fill(samples_.begin(), samples_.end(), base::TimeDelta());
39 }
40
41 } // namespace media
OLDNEW
« no previous file with comments | « media/base/moving_average.h ('k') | media/base/moving_average_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698