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

Side by Side Diff: chrome/browser/chromeos/power/power_data_collector.cc

Issue 196613002: Revert of [chromeos/about:power] Collect cpuidle and cpufreq stats (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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 | Annotate | Revision Log
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 "chrome/browser/chromeos/power/power_data_collector.h" 5 #include "chrome/browser/chromeos/power/power_data_collector.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "chromeos/dbus/dbus_thread_manager.h" 8 #include "chromeos/dbus/dbus_thread_manager.h"
9 #include "chromeos/dbus/power_manager/power_supply_properties.pb.h" 9 #include "chromeos/dbus/power_manager/power_supply_properties.pb.h"
10 10
11 namespace chromeos { 11 namespace chromeos {
12 12
13 namespace { 13 namespace {
14 14
15 // The global PowerDataCollector instance. 15 // The global PowerDataCollector instance.
16 PowerDataCollector* g_power_data_collector = NULL; 16 PowerDataCollector* g_power_data_collector = NULL;
17 17
18 } // namespace 18 } // namespace
19 19
20 const int PowerDataCollector::kSampleTimeLimitSec = 24 * 60 * 60; 20 const int PowerDataCollector::kSampleTimeLimitSec = 24 * 60 * 60;
21 21
22 // static 22 // static
23 void PowerDataCollector::Initialize() { 23 void PowerDataCollector::Initialize() {
24 // Check that power data collector is initialized only after the 24 // Check that power data collector is initialized only after the
25 // DBusThreadManager is initialized. 25 // DBusThreadManager is initialized.
26 CHECK(DBusThreadManager::Get()); 26 CHECK(DBusThreadManager::Get());
27 CHECK(g_power_data_collector == NULL); 27 CHECK(g_power_data_collector == NULL);
28 g_power_data_collector = new PowerDataCollector(true); 28 g_power_data_collector = new PowerDataCollector();
29 }
30
31 void PowerDataCollector::InitializeForTesting() {
32 CHECK(DBusThreadManager::Get());
33 CHECK(g_power_data_collector == NULL);
34 g_power_data_collector = new PowerDataCollector(false);
35 } 29 }
36 30
37 // static 31 // static
38 void PowerDataCollector::Shutdown() { 32 void PowerDataCollector::Shutdown() {
39 // Shutdown only if initialized. 33 // Shutdown only if initialized.
40 CHECK(g_power_data_collector); 34 CHECK(g_power_data_collector);
41 delete g_power_data_collector; 35 delete g_power_data_collector;
42 g_power_data_collector = NULL; 36 g_power_data_collector = NULL;
43 } 37 }
44 38
(...skipping 14 matching lines...) Expand all
59 AddSample(&power_supply_data_, sample); 53 AddSample(&power_supply_data_, sample);
60 } 54 }
61 55
62 void PowerDataCollector::SystemResumed(const base::TimeDelta& sleep_duration) { 56 void PowerDataCollector::SystemResumed(const base::TimeDelta& sleep_duration) {
63 SystemResumedSample sample; 57 SystemResumedSample sample;
64 sample.time = base::Time::Now(); 58 sample.time = base::Time::Now();
65 sample.sleep_duration = sleep_duration; 59 sample.sleep_duration = sleep_duration;
66 AddSample(&system_resumed_data_, sample); 60 AddSample(&system_resumed_data_, sample);
67 } 61 }
68 62
69 PowerDataCollector::PowerDataCollector(const bool start_cpu_data_collector) { 63 PowerDataCollector::PowerDataCollector() {
70 DBusThreadManager::Get()->GetPowerManagerClient()->AddObserver(this); 64 DBusThreadManager::Get()->GetPowerManagerClient()->AddObserver(this);
71 if (start_cpu_data_collector)
72 cpu_data_collector_.Start();
73 } 65 }
74 66
75 PowerDataCollector::~PowerDataCollector() { 67 PowerDataCollector::~PowerDataCollector() {
76 DBusThreadManager* dbus_manager = DBusThreadManager::Get(); 68 DBusThreadManager* dbus_manager = DBusThreadManager::Get();
77 CHECK(dbus_manager); 69 CHECK(dbus_manager);
78 dbus_manager->GetPowerManagerClient()->RemoveObserver(this); 70 dbus_manager->GetPowerManagerClient()->RemoveObserver(this);
79 } 71 }
80 72
81 PowerDataCollector::PowerSupplySample::PowerSupplySample() 73 PowerDataCollector::PowerSupplySample::PowerSupplySample()
82 : external_power(false), 74 : external_power(false),
83 battery_percent(0.0), 75 battery_percent(0.0),
84 battery_discharge_rate(0.0) { 76 battery_discharge_rate(0.0) {
85 } 77 }
86 78
87 PowerDataCollector::SystemResumedSample::SystemResumedSample() { 79 PowerDataCollector::SystemResumedSample::SystemResumedSample() {
88 } 80 }
89 81
90 } // namespace chromeos 82 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/power/power_data_collector.h ('k') | chrome/browser/chromeos/power/power_data_collector_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698