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

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

Issue 667943003: Standardize usage of virtual/override/final in content/browser/ (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
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 ~FakeBatteryManager() override {}
22 22
23 // Methods from Battery Status Manager 23 // Methods from Battery Status Manager
24 virtual bool StartListeningBatteryChange() override { 24 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 void StopListeningBatteryChange() override { stop_invoked_count_++; }
30 stop_invoked_count_++;
31 }
32 30
33 void InvokeUpdateCallback(const blink::WebBatteryStatus& status) { 31 void InvokeUpdateCallback(const blink::WebBatteryStatus& status) {
34 callback_.Run(status); 32 callback_.Run(status);
35 } 33 }
36 34
37 int start_invoked_count() const { return start_invoked_count_; } 35 int start_invoked_count() const { return start_invoked_count_; }
38 int stop_invoked_count() const { return stop_invoked_count_; } 36 int stop_invoked_count() const { return stop_invoked_count_; }
39 37
40 private: 38 private:
41 BatteryStatusService::BatteryUpdateCallback callback_; 39 BatteryStatusService::BatteryUpdateCallback callback_;
(...skipping 140 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 subscription1.reset(); 180 subscription1.reset();
183 battery_manager()->InvokeUpdateCallback(status); 181 battery_manager()->InvokeUpdateCallback(status);
184 base::RunLoop().RunUntilIdle(); 182 base::RunLoop().RunUntilIdle();
185 EXPECT_EQ(1, callback1_invoked_count()); 183 EXPECT_EQ(1, callback1_invoked_count());
186 EXPECT_EQ(2, callback2_invoked_count()); 184 EXPECT_EQ(2, callback2_invoked_count());
187 } 185 }
188 186
189 } // namespace 187 } // namespace
190 188
191 } // namespace content 189 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/battery_status/battery_status_message_filter.h ('k') | content/browser/bootstrap_sandbox_mac.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698