From 8ab89c5bfe31fe3237b0307e50bb301688166f7e Mon Sep 17 00:00:00 2001 From: Steve Fung Date: Mon, 5 Jan 2015 13:48:30 -0800 Subject: [PATCH] metrics: Fix metrics_daemon Meminfo callback interval With the switch from the glib run loop to base::MessageLoop, the interval for scheduling Meminfo callbacks was incorrectly set to milliseconds. Fix it back to seconds. BUG=chromium:445573 TEST=strace shows metrics_daemon sleep for 30 seconds between collecting statistics Change-Id: I4b4b597273d3cf04b9972011dd0a7386ea14233d Reviewed-on: https://chromium-review.googlesource.com/238466 Reviewed-by: Chih-Chung Chang Reviewed-by: Alex Vakulenko Commit-Queue: Steve Fung Tested-by: Steve Fung --- metrics/metrics_daemon.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/metrics/metrics_daemon.cc b/metrics/metrics_daemon.cc index acd96c079..a746e06a0 100644 --- a/metrics/metrics_daemon.cc +++ b/metrics/metrics_daemon.cc @@ -776,7 +776,7 @@ void MetricsDaemon::ScheduleMeminfoCallback(int wait) { base::TimeDelta waitDelta = base::TimeDelta::FromSeconds(wait); base::MessageLoop::current()->PostDelayedTask(FROM_HERE, base::Bind(&MetricsDaemon::MeminfoCallback, base::Unretained(this), - base::TimeDelta::FromMilliseconds(kMetricMeminfoInterval)), + waitDelta), waitDelta); } @@ -795,7 +795,7 @@ void MetricsDaemon::MeminfoCallback(base::TimeDelta wait) { if (reschedule) { base::MessageLoop::current()->PostDelayedTask(FROM_HERE, base::Bind(&MetricsDaemon::MeminfoCallback, base::Unretained(this), - base::TimeDelta::FromMilliseconds(kMetricMeminfoInterval)), + wait), wait); } }