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

Side by Side Diff: components/network_time/network_time_tracker_unittest.cc

Issue 684513002: Standardize usage of virtual/override/final specifiers. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 6 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 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/network_time/network_time_tracker.h" 5 #include "components/network_time/network_time_tracker.h"
6 6
7 #include <math.h> 7 #include <math.h>
8 8
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/prefs/testing_pref_service.h" 10 #include "base/prefs/testing_pref_service.h"
(...skipping 24 matching lines...) Expand all
35 base::TimeTicks NowTicks() override { return *ticks_now_; } 35 base::TimeTicks NowTicks() override { return *ticks_now_; }
36 36
37 private: 37 private:
38 base::TimeTicks* ticks_now_; 38 base::TimeTicks* ticks_now_;
39 }; 39 };
40 40
41 } // namespace 41 } // namespace
42 42
43 class NetworkTimeTrackerTest : public testing::Test { 43 class NetworkTimeTrackerTest : public testing::Test {
44 public: 44 public:
45 virtual ~NetworkTimeTrackerTest() {} 45 ~NetworkTimeTrackerTest() override {}
46 46
47 virtual void SetUp() override { 47 void SetUp() override {
48 NetworkTimeTracker::RegisterPrefs(pref_service_.registry()); 48 NetworkTimeTracker::RegisterPrefs(pref_service_.registry());
49 49
50 now_ = base::Time::NowFromSystemTime(); 50 now_ = base::Time::NowFromSystemTime();
51 network_time_tracker_.reset(new NetworkTimeTracker( 51 network_time_tracker_.reset(new NetworkTimeTracker(
52 scoped_ptr<base::TickClock>(new TestTickClock(&ticks_now_)), 52 scoped_ptr<base::TickClock>(new TestTickClock(&ticks_now_)),
53 &pref_service_)); 53 &pref_service_));
54 } 54 }
55 55
56 base::Time Now() const { 56 base::Time Now() const {
57 return now_ + (ticks_now_ - base::TimeTicks()); 57 return now_ + (ticks_now_ - base::TimeTicks());
(...skipping 94 matching lines...) Expand 10 before | Expand all | Expand 10 after
152 AddToTicksNow(kPseudoSleepTime2); 152 AddToTicksNow(kPseudoSleepTime2);
153 UpdateNetworkTime( 153 UpdateNetworkTime(
154 old_now, 154 old_now,
155 base::TimeDelta::FromMilliseconds(kResolution2), 155 base::TimeDelta::FromMilliseconds(kResolution2),
156 base::TimeDelta::FromMilliseconds(kLatency2), 156 base::TimeDelta::FromMilliseconds(kLatency2),
157 old_ticks); 157 old_ticks);
158 EXPECT_TRUE(ValidateExpectedTime()); 158 EXPECT_TRUE(ValidateExpectedTime());
159 } 159 }
160 160
161 } // namespace network_time 161 } // namespace network_time
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698