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

Unified Diff: chrome/browser/ui/webui/chromeos/power_ui.cc

Issue 134623002: [chromeos] Limit the data samples stored by PowerDataCollector. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | chromeos/power/power_data_collector.h » ('j') | chromeos/power/power_data_collector.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/webui/chromeos/power_ui.cc
diff --git a/chrome/browser/ui/webui/chromeos/power_ui.cc b/chrome/browser/ui/webui/chromeos/power_ui.cc
index 92ef8d50a8f5e81bb2eb01a86d45dee5c22422b5..0dc0bec0c6dec813eb596565e8a4d7ac00a0be5b 100644
--- a/chrome/browser/ui/webui/chromeos/power_ui.cc
+++ b/chrome/browser/ui/webui/chromeos/power_ui.cc
@@ -54,14 +54,16 @@ void PowerMessageHandler::RegisterMessages() {
}
void PowerMessageHandler::OnGetBatteryChargeData(const base::ListValue* value) {
- const std::vector<PowerDataCollector::PowerSupplySnapshot>& power_supply =
+ const std::list<PowerDataCollector::PowerSupplySnapshot>& power_supply =
PowerDataCollector::Get()->power_supply_data();
base::ListValue data;
+ typename std::list<PowerDataCollector::PowerSupplySnapshot>::const_iterator
+ iter;
- for (unsigned int i = 0; i < power_supply.size(); ++i) {
- const PowerDataCollector::PowerSupplySnapshot& snapshot = power_supply[i];
+ for (iter = power_supply.begin(); iter != power_supply.end(); ++iter) {
+ const PowerDataCollector::PowerSupplySnapshot& snapshot = *iter;
base::Time time = base::Time::Now() -
- (base::TimeTicks::Now() - power_supply[i].time);
+ (base::TimeTicks::Now() - snapshot.time);
scoped_ptr<base::DictionaryValue> element(new base::DictionaryValue);
element->SetDouble("battery_percent", snapshot.battery_percent);
element->SetBoolean("external_power", snapshot.external_power);
« no previous file with comments | « no previous file | chromeos/power/power_data_collector.h » ('j') | chromeos/power/power_data_collector.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698