From adc1c235378f6d80a5c11d5a1197524d5cd0bf51 Mon Sep 17 00:00:00 2001 From: Nathan Bullock Date: Fri, 6 Nov 2015 14:02:45 -0500 Subject: [PATCH] metricsd: add SendBoolToUMA to metrics interface Bug: 25561571 Change-Id: I698d2408e2f462865f6512b7be17ff0851a1f97c --- metricsd/include/metrics/metrics_library.h | 4 ++++ metricsd/include/metrics/metrics_library_mock.h | 1 + metricsd/metrics_library.cc | 7 +++++++ 3 files changed, 12 insertions(+) diff --git a/metricsd/include/metrics/metrics_library.h b/metricsd/include/metrics/metrics_library.h index b76619456..d2e98c8df 100644 --- a/metricsd/include/metrics/metrics_library.h +++ b/metricsd/include/metrics/metrics_library.h @@ -34,6 +34,7 @@ class MetricsLibraryInterface { virtual bool SendToUMA(const std::string& name, int sample, int min, int max, int nbuckets) = 0; virtual bool SendEnumToUMA(const std::string& name, int sample, int max) = 0; + virtual bool SendBoolToUMA(const std::string& name, bool sample) = 0; virtual bool SendSparseToUMA(const std::string& name, int sample) = 0; virtual bool SendUserActionToUMA(const std::string& action) = 0; virtual ~MetricsLibraryInterface() {} @@ -96,6 +97,9 @@ class MetricsLibrary : public MetricsLibraryInterface { // normal, while 100 is high). bool SendEnumToUMA(const std::string& name, int sample, int max) override; + // Specialization of SendEnumToUMA for boolean values. + bool SendBoolToUMA(const std::string& name, bool sample) override; + // Sends sparse histogram sample to Chrome for transport to UMA. Returns // true on success. // diff --git a/metricsd/include/metrics/metrics_library_mock.h b/metricsd/include/metrics/metrics_library_mock.h index 3de87a9f0..db56f9e50 100644 --- a/metricsd/include/metrics/metrics_library_mock.h +++ b/metricsd/include/metrics/metrics_library_mock.h @@ -32,6 +32,7 @@ class MetricsLibraryMock : public MetricsLibraryInterface { int min, int max, int nbuckets)); MOCK_METHOD3(SendEnumToUMA, bool(const std::string& name, int sample, int max)); + MOCK_METHOD2(SendBoolToUMA, bool(const std::string& name, bool sample)); MOCK_METHOD2(SendSparseToUMA, bool(const std::string& name, int sample)); MOCK_METHOD1(SendUserActionToUMA, bool(const std::string& action)); diff --git a/metricsd/metrics_library.cc b/metricsd/metrics_library.cc index a651b7676..735d39ff8 100644 --- a/metricsd/metrics_library.cc +++ b/metricsd/metrics_library.cc @@ -173,6 +173,13 @@ bool MetricsLibrary::SendEnumToUMA(const std::string& name, int sample, uma_events_file_.value()); } +bool MetricsLibrary::SendBoolToUMA(const std::string& name, bool sample) { + return metrics::SerializationUtils::WriteMetricToFile( + *metrics::MetricSample::LinearHistogramSample(name, + sample ? 1 : 0, 2).get(), + uma_events_file_.value()); +} + bool MetricsLibrary::SendSparseToUMA(const std::string& name, int sample) { return metrics::SerializationUtils::WriteMetricToFile( *metrics::MetricSample::SparseHistogramSample(name, sample).get(),