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

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

Powered by Google App Engine
This is Rietveld 408576698