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

Side by Side Diff: content/browser/power_usage_monitor_impl_unittest.cc

Issue 2456783002: Decouple PowerUsageMonitor{Impl} from //content/browser (Closed)
Patch Set: change define position of power_usage_moniter_ in ChromeBrowserMain Created 4 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "content/browser/power_usage_monitor_impl.h"
6
7 #include <utility>
8
9 #include "content/public/browser/notification_types.h"
10 #include "content/public/test/test_browser_thread_bundle.h"
11 #include "device/battery/battery_monitor.mojom.h"
12 #include "testing/gtest/include/gtest/gtest.h"
13
14 namespace content {
15
16 // Dummy ID to identify a phantom RenderProcessHost in tests.
17 const int kDummyRenderProcessHostID = 1;
18
19 class SystemInterfaceForTest : public PowerUsageMonitor::SystemInterface {
20 public:
21 SystemInterfaceForTest()
22 : num_pending_histogram_reports_(0),
23 discharge_percent_per_hour_(0),
24 now_(base::Time::FromInternalValue(1000)) {}
25 ~SystemInterfaceForTest() override {}
26
27 int num_pending_histogram_reports() const {
28 return num_pending_histogram_reports_;
29 }
30
31 int discharge_percent_per_hour() const {
32 return discharge_percent_per_hour_;
33 }
34
35 void AdvanceClockSeconds(int seconds) {
36 now_ += base::TimeDelta::FromSeconds(seconds);
37 }
38
39 void AdvanceClockMinutes(int minutes) {
40 now_ += base::TimeDelta::FromMinutes(minutes);
41 }
42
43 void ScheduleHistogramReport(base::TimeDelta delay) override {
44 num_pending_histogram_reports_++;
45 }
46
47 void CancelPendingHistogramReports() override {
48 num_pending_histogram_reports_ = 0;
49 }
50
51 void RecordDischargePercentPerHour(int percent_per_hour) override {
52 discharge_percent_per_hour_ = percent_per_hour;
53 }
54
55 base::Time Now() override { return now_; }
56
57 private:
58 int num_pending_histogram_reports_;
59 int discharge_percent_per_hour_;
60 base::Time now_;
61 };
62
63 class PowerUsageMonitorTest : public testing::Test {
64 protected:
65 void SetUp() override {
66 monitor_.reset(new PowerUsageMonitor);
67 // PowerUsageMonitor assumes ownership.
68 std::unique_ptr<SystemInterfaceForTest> test_interface(
69 new SystemInterfaceForTest());
70 system_interface_ = test_interface.get();
71 monitor_->SetSystemInterfaceForTest(std::move(test_interface));
72
73 // Without live renderers, the monitor won't do anything.
74 monitor_->OnRenderProcessNotification(NOTIFICATION_RENDERER_PROCESS_CREATED,
75 kDummyRenderProcessHostID);
76 }
77
78 void UpdateBatteryStatus(bool charging, double battery_level) {
79 device::BatteryStatus battery_status;
80 battery_status.charging = charging;
81 battery_status.level = battery_level;
82 monitor_->OnBatteryStatusUpdate(battery_status);
83 }
84
85 void KillTestRenderer() {
86 monitor_->OnRenderProcessNotification(
87 NOTIFICATION_RENDERER_PROCESS_CLOSED, kDummyRenderProcessHostID);
88 }
89
90 std::unique_ptr<PowerUsageMonitor> monitor_;
91 SystemInterfaceForTest* system_interface_;
92 TestBrowserThreadBundle thread_bundle_;
93 };
94
95 TEST_F(PowerUsageMonitorTest, StartStopQuickly) {
96 // Going on battery power.
97 UpdateBatteryStatus(false, 1.0);
98 int initial_num_histogram_reports =
99 system_interface_->num_pending_histogram_reports();
100 ASSERT_GT(initial_num_histogram_reports, 0);
101
102 // Battery level goes down a bit.
103 system_interface_->AdvanceClockSeconds(1);
104 UpdateBatteryStatus(false, 0.9);
105 ASSERT_EQ(initial_num_histogram_reports,
106 system_interface_->num_pending_histogram_reports());
107 ASSERT_EQ(0, system_interface_->discharge_percent_per_hour());
108
109 // Wall power connected.
110 system_interface_->AdvanceClockSeconds(30);
111 UpdateBatteryStatus(true, 0);
112 ASSERT_EQ(0, system_interface_->num_pending_histogram_reports());
113 ASSERT_EQ(0, system_interface_->discharge_percent_per_hour());
114 }
115
116 TEST_F(PowerUsageMonitorTest, DischargePercentReported) {
117 // Going on battery power.
118 UpdateBatteryStatus(false, 1.0);
119 int initial_num_histogram_reports =
120 system_interface_->num_pending_histogram_reports();
121 ASSERT_GT(initial_num_histogram_reports, 0);
122
123 // Battery level goes down a bit.
124 system_interface_->AdvanceClockSeconds(30);
125 UpdateBatteryStatus(false, 0.9);
126 ASSERT_EQ(initial_num_histogram_reports,
127 system_interface_->num_pending_histogram_reports());
128 ASSERT_EQ(0, system_interface_->discharge_percent_per_hour());
129
130 // Wall power connected.
131 system_interface_->AdvanceClockMinutes(31);
132 UpdateBatteryStatus(true, 0);
133 ASSERT_EQ(0, system_interface_->num_pending_histogram_reports());
134 ASSERT_GT(system_interface_->discharge_percent_per_hour(), 0);
135 }
136
137 TEST_F(PowerUsageMonitorTest, NoRenderersDisablesMonitoring) {
138 KillTestRenderer();
139
140 // Going on battery power.
141 UpdateBatteryStatus(false, 1.0);
142 ASSERT_EQ(0, system_interface_->num_pending_histogram_reports());
143 ASSERT_EQ(0, system_interface_->discharge_percent_per_hour());
144
145 // Wall power connected.
146 system_interface_->AdvanceClockSeconds(30);
147 UpdateBatteryStatus(true, 0.5);
148 ASSERT_EQ(0, system_interface_->num_pending_histogram_reports());
149 ASSERT_EQ(0, system_interface_->discharge_percent_per_hour());
150 }
151
152 TEST_F(PowerUsageMonitorTest, NoRenderersCancelsInProgressMonitoring) {
153 // Going on battery power.
154 UpdateBatteryStatus(false, 1.0);
155 ASSERT_GT(system_interface_->num_pending_histogram_reports(), 0);
156 ASSERT_EQ(0, system_interface_->discharge_percent_per_hour());
157
158 // All renderers killed.
159 KillTestRenderer();
160 ASSERT_EQ(0, system_interface_->num_pending_histogram_reports());
161
162 // Wall power connected.
163 system_interface_->AdvanceClockMinutes(31);
164 UpdateBatteryStatus(true, 0);
165 ASSERT_EQ(0, system_interface_->num_pending_histogram_reports());
166 ASSERT_EQ(0, system_interface_->discharge_percent_per_hour());
167 }
168
169 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/power_usage_monitor_impl.cc ('k') | content/public/browser/power_usage_monitor.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698