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

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

Issue 560553005: Battery impact UMA (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Now with unit tests 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
(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 "testing/gtest/include/gtest/gtest.h"
8
9 namespace content {
10
11 class SystemInterfaceForTest : public PowerUsageMonitor::SystemInterface {
12 public:
13 SystemInterfaceForTest()
14 : num_pending_histogram_reports_(0), now_(base::Time::Now()) {}
15 virtual ~SystemInterfaceForTest() {}
16
17 int num_pending_histogram_reports() { return num_pending_histogram_reports_; }
18
19 virtual void AdvanceClockSeconds(int seconds) {
20 now_ += base::TimeDelta::FromSeconds(seconds);
21 }
22
23 virtual void AdvanceClockHours(int hours) {
24 now_ += base::TimeDelta::FromHours(hours);
25 }
26
27 virtual void ScheduleHistogramReport(base::TimeDelta delay) override {
28 num_pending_histogram_reports_++;
29 }
30
31 virtual void CancelPendingHistgramReports() override {
32 num_pending_histogram_reports_ = 0;
33 }
34
35 virtual base::Time Now() override { return now_; }
36
37 protected:
38 virtual void ReportBatteryLevelHistogram(
39 base::TimeDelta discharge_time) override {}
40
41 private:
42 int num_pending_histogram_reports_;
43 base::Time now_;
44 };
45
46 class PowerUsageMonitorTest : public testing::Test {
47 protected:
48 virtual void SetUp() {
49 monitor_.reset(new PowerUsageMonitor);
50 // PowerUsageMonitor assumes ownership.
51 scoped_ptr<SystemInterfaceForTest> test_interface(
52 new SystemInterfaceForTest());
53 system_interface_ = test_interface.get();
54 monitor_->SetSystemInterfaceForTest(test_interface.Pass());
55
56 // Without live renderers, the monitor won't do anything.
57 monitor_->SetNumLiveRenderersForTest(1);
58 }
59
60 void UpdateBatteryStatus(bool charging, double battery_level) {
61 blink::WebBatteryStatus battery_status;
62 battery_status.charging = charging;
63 battery_status.level = battery_level;
64 monitor_->OnBatteryStatusUpdate(battery_status);
65 }
66 scoped_ptr<PowerUsageMonitor> monitor_;
67 SystemInterfaceForTest* system_interface_;
68 };
69
70 TEST_F(PowerUsageMonitorTest, StartStop) {
71 // Going on battery power.
72 UpdateBatteryStatus(false, 1.0);
73 int initial_num_histogram_reports =
74 system_interface_->num_pending_histogram_reports();
75 ASSERT_GT(initial_num_histogram_reports, 0);
76
77 // Battery level goes down a bit.
78 system_interface_->AdvanceClockSeconds(1);
79 UpdateBatteryStatus(false, 0.9);
80 ASSERT_EQ(initial_num_histogram_reports,
81 system_interface_->num_pending_histogram_reports());
82
83 // Wall power connected.
84 system_interface_->AdvanceClockSeconds(30);
85 UpdateBatteryStatus(true, 0.5);
86 ASSERT_EQ(system_interface_->num_pending_histogram_reports(), 0);
87 }
88
89 TEST_F(PowerUsageMonitorTest, NoRenderersDisablesMonitoring) {
90 monitor_->SetNumLiveRenderersForTest(0);
91
92 // Going on battery power.
93 UpdateBatteryStatus(false, 1.0);
94 ASSERT_EQ(system_interface_->num_pending_histogram_reports(), 0);
95
96 // Wall power connected.
97 system_interface_->AdvanceClockSeconds(30);
98 UpdateBatteryStatus(true, 0.5);
99 ASSERT_EQ(system_interface_->num_pending_histogram_reports(), 0);
100 }
101
102 TEST_F(PowerUsageMonitorTest, ReportOnceEvery24Hours) {
103 // Going on battery power.
104 UpdateBatteryStatus(false, 1.0);
105 ASSERT_GT(system_interface_->num_pending_histogram_reports(), 0);
106
107 // Tell the monitor that a histogram was recorded.
108 monitor_->BatteryLevelReported();
109
110 // Connect back to wall power.
111 system_interface_->AdvanceClockHours(1);
112 UpdateBatteryStatus(true, 0.9);
113
114 // Disconnect from Wall power.
115 UpdateBatteryStatus(false, 1.0);
116 ASSERT_EQ(system_interface_->num_pending_histogram_reports(), 0);
117
118 // Back on wall power.
119 UpdateBatteryStatus(true, 0.9);
120
121 // 24 hours later.
122 system_interface_->AdvanceClockHours(24);
123
124 // Disconnect from Wall power.
125 UpdateBatteryStatus(false, 1.0);
126 ASSERT_GT(system_interface_->num_pending_histogram_reports(), 0);
127 }
128
129 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698