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

Side by Side Diff: content/browser/battery_status/battery_status_service_unittest.cc

Issue 633543002: Replace OVERRIDE and FINAL with override and final in content/browser/battery_status[a-s]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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
« no previous file with comments | « content/browser/battery_status/battery_status_message_filter.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "content/browser/battery_status/battery_status_service.h" 5 #include "content/browser/battery_status/battery_status_service.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/run_loop.h" 8 #include "base/run_loop.h"
9 #include "content/browser/battery_status/battery_status_manager.h" 9 #include "content/browser/battery_status/battery_status_manager.h"
10 #include "content/public/test/test_browser_thread_bundle.h" 10 #include "content/public/test/test_browser_thread_bundle.h"
11 11
12 namespace content { 12 namespace content {
13 13
14 namespace { 14 namespace {
15 15
16 class FakeBatteryManager : public BatteryStatusManager { 16 class FakeBatteryManager : public BatteryStatusManager {
17 public: 17 public:
18 explicit FakeBatteryManager( 18 explicit FakeBatteryManager(
19 const BatteryStatusService::BatteryUpdateCallback& callback) 19 const BatteryStatusService::BatteryUpdateCallback& callback)
20 : callback_(callback), start_invoked_count_(0), stop_invoked_count_(0) {} 20 : callback_(callback), start_invoked_count_(0), stop_invoked_count_(0) {}
21 virtual ~FakeBatteryManager() { } 21 virtual ~FakeBatteryManager() { }
22 22
23 // Methods from Battery Status Manager 23 // Methods from Battery Status Manager
24 virtual bool StartListeningBatteryChange() OVERRIDE { 24 virtual bool StartListeningBatteryChange() override {
25 start_invoked_count_++; 25 start_invoked_count_++;
26 return true; 26 return true;
27 } 27 }
28 28
29 virtual void StopListeningBatteryChange() OVERRIDE { 29 virtual void StopListeningBatteryChange() override {
30 stop_invoked_count_++; 30 stop_invoked_count_++;
31 } 31 }
32 32
33 void InvokeUpdateCallback(const blink::WebBatteryStatus& status) { 33 void InvokeUpdateCallback(const blink::WebBatteryStatus& status) {
34 callback_.Run(status); 34 callback_.Run(status);
35 } 35 }
36 36
37 int start_invoked_count() const { return start_invoked_count_; } 37 int start_invoked_count() const { return start_invoked_count_; }
38 int stop_invoked_count() const { return stop_invoked_count_; } 38 int stop_invoked_count() const { return stop_invoked_count_; }
39 39
(...skipping 12 matching lines...) Expand all
52 battery_service_(0), 52 battery_service_(0),
53 battery_manager_(0), 53 battery_manager_(0),
54 callback1_invoked_count_(0), 54 callback1_invoked_count_(0),
55 callback2_invoked_count_(0) { 55 callback2_invoked_count_(0) {
56 } 56 }
57 virtual ~BatteryStatusServiceTest() { } 57 virtual ~BatteryStatusServiceTest() { }
58 58
59 protected: 59 protected:
60 typedef BatteryStatusService::BatteryUpdateSubscription BatterySubscription; 60 typedef BatteryStatusService::BatteryUpdateSubscription BatterySubscription;
61 61
62 virtual void SetUp() OVERRIDE { 62 virtual void SetUp() override {
63 callback1_ = base::Bind(&BatteryStatusServiceTest::Callback1, 63 callback1_ = base::Bind(&BatteryStatusServiceTest::Callback1,
64 base::Unretained(this)); 64 base::Unretained(this));
65 callback2_ = base::Bind(&BatteryStatusServiceTest::Callback2, 65 callback2_ = base::Bind(&BatteryStatusServiceTest::Callback2,
66 base::Unretained(this)); 66 base::Unretained(this));
67 battery_service_ = BatteryStatusService::GetInstance(); 67 battery_service_ = BatteryStatusService::GetInstance();
68 battery_manager_ = new FakeBatteryManager( 68 battery_manager_ = new FakeBatteryManager(
69 battery_service_->GetUpdateCallbackForTesting()); 69 battery_service_->GetUpdateCallbackForTesting());
70 battery_service_->SetBatteryManagerForTesting(battery_manager_); 70 battery_service_->SetBatteryManagerForTesting(battery_manager_);
71 } 71 }
72 72
73 virtual void TearDown() OVERRIDE { 73 virtual void TearDown() override {
74 base::RunLoop().RunUntilIdle(); 74 base::RunLoop().RunUntilIdle();
75 battery_service_->SetBatteryManagerForTesting(0); 75 battery_service_->SetBatteryManagerForTesting(0);
76 } 76 }
77 77
78 FakeBatteryManager* battery_manager() { 78 FakeBatteryManager* battery_manager() {
79 return battery_manager_; 79 return battery_manager_;
80 } 80 }
81 81
82 scoped_ptr<BatterySubscription> AddCallback( 82 scoped_ptr<BatterySubscription> AddCallback(
83 const BatteryStatusService::BatteryUpdateCallback& callback) { 83 const BatteryStatusService::BatteryUpdateCallback& callback) {
(...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 subscription1.reset(); 182 subscription1.reset();
183 battery_manager()->InvokeUpdateCallback(status); 183 battery_manager()->InvokeUpdateCallback(status);
184 base::RunLoop().RunUntilIdle(); 184 base::RunLoop().RunUntilIdle();
185 EXPECT_EQ(1, callback1_invoked_count()); 185 EXPECT_EQ(1, callback1_invoked_count());
186 EXPECT_EQ(2, callback2_invoked_count()); 186 EXPECT_EQ(2, callback2_invoked_count());
187 } 187 }
188 188
189 } // namespace 189 } // namespace
190 190
191 } // namespace content 191 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/battery_status/battery_status_message_filter.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698