commit 4a6c942d1d3bae7ba8dfe67e1487b66bc8c311c7
author Luigi Semenzato <semenzato@chromium.org> Mon Jun 30 18:12:28 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Jul 02 22:11:43 2014 +0000
tree f77b83099430d1a03f16adc5bd6fe7e41e03846a
parent 6a26a0d4c69528ddd827357ecf2dc4bd95ee42d8

metrics_daemon: fix parsing of zram sysfs

The only real change is in ReadFileToUint64.  The other changes
appease the updated pre-commit checks.

BUG=chromium:390334
TEST=ran manually, checked syslog, checked chrome://histograms

Change-Id: I5ca10aab03697aa8fd64c5dd51ca64911e8d2a76
Reviewed-on: https://chromium-review.googlesource.com/206285
Tested-by: Luigi Semenzato <semenzato@chromium.org>
Reviewed-by: Daniel Erat <derat@chromium.org>
Commit-Queue: Luigi Semenzato <semenzato@chromium.org>

metrics/metrics_daemon.h

diff --git a/metrics/metrics_daemon.h b/metrics/metrics_daemon.h
index df85a2f..9c234a1 100644
--- a/metrics/metrics_daemon.h
+++ b/metrics/metrics_daemon.h
@@ -2,25 +2,26 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef METRICS_DAEMON_H_
-#define METRICS_DAEMON_H_
+#ifndef METRICS_METRICS_DAEMON_H_
+#define METRICS_METRICS_DAEMON_H_
 
 #include <dbus/dbus.h>
 #include <glib.h>
 #include <map>
+#include <string>
+#include <vector>
 
 #include <base/files/file_path.h>
 #include <base/memory/scoped_ptr.h>
 #include <base/time/time.h>
 #include <gtest/gtest_prod.h>  // for FRIEND_TEST
 
-#include "metrics_library.h"
-#include "persistent_integer.h"
+#include "metrics/metrics_library.h"
+#include "metrics/persistent_integer.h"
 
 using chromeos_metrics::PersistentInteger;
 
 class MetricsDaemon {
-
  public:
   MetricsDaemon();
   ~MetricsDaemon();
@@ -206,7 +207,7 @@
   void ScheduleStatsCallback(int wait);
 
   // Reads cumulative disk statistics from sysfs.  Returns true for success.
-  bool DiskStatsReadStats(long int* read_sectors, long int* write_sectors);
+  bool DiskStatsReadStats(uint64* read_sectors, uint64* write_sectors);
 
   // Reads cumulative vm statistics from procfs.  Returns true for success.
   bool VmStatsReadStats(struct VmstatRecord* stats);
@@ -316,8 +317,8 @@
   unsigned int memuse_interval_index_;
 
   // Contain the most recent disk and vm cumulative stats.
-  long int read_sectors_;
-  long int write_sectors_;
+  uint64 read_sectors_;
+  uint64 write_sectors_;
   struct VmstatRecord vmstats_;
 
   StatsState stats_state_;
@@ -364,4 +365,4 @@
   std::string cpuinfo_max_freq_path_;
 };
 
-#endif  // METRICS_DAEMON_H_
+#endif  // METRICS_METRICS_DAEMON_H_