metrics cleanup and fixes.

src/platform/metrics/metrics_daemon.cc

 // Copyright (c) 2010 The Chromium OS Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "metrics_daemon.h"
 #include "metrics_library.h"
 
 #include <glib-object.h>
 
 extern "C" {
@@ skipping 95 matching lines @@
   }
   if (old_id != kUnknownNetworkStateId) {
     struct timeval diff;
     timersub(&now, &network_state_start_, &diff);
     int diff_ms = diff.tv_usec / 1000 + diff.tv_sec * 1000;
     // Saturates rather than overflowing.  We expect this to be statistically
     // insignificant, since INT_MAX milliseconds is 24.8 days.
     if (diff.tv_sec >= INT_MAX / 1000) {
       diff_ms = INT_MAX;
     }
-    char buffer[100];
-    snprintf(buffer, sizeof(buffer), "%d", diff_ms);
     if (testing_) {
-      TestPublishMetric(network_states_[old_id].stat_name, buffer);
+      TestPublishMetric(network_states_[old_id].stat_name, diff_ms);
     } else {
-      ChromePublishMetric(network_states_[old_id].stat_name, buffer);
+      ChromePublishMetric(network_states_[old_id].stat_name, diff_ms);
     }
   }
   network_state_id_ = new_id;
   network_state_start_ = now;
 }
 
 MetricsDaemon::NetworkStateId
 MetricsDaemon::GetNetworkStateId(const char* state_name) {
   for (int i = 0; i < kNumberNetworkStates; i++) {
     if (strcmp(state_name, network_states_[i].name) == 0) {
       return static_cast<NetworkStateId>(i);
     }
   }
   return static_cast<NetworkStateId>(-1);
 }
 
-void MetricsDaemon::ChromePublishMetric(const char* name, const char* value) {
+void MetricsDaemon::ChromePublishMetric(const char* name, int value) {
   MetricsLibrary::SendToChrome(name, value);
 }
 
-void MetricsDaemon::TestPublishMetric(const char* name, const char* value) {
+void MetricsDaemon::TestPublishMetric(const char* name, int value) {
   LOG(INFO) << "received metric: " << name << " " << value;
 }