[Chromecast] Add metric reporting to ConnectivityCheckerImpl

chromecast/base/metrics/cast_metrics_helper.cc

 // Copyright 2014 The Chromium 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 "chromecast/base/metrics/cast_metrics_helper.h"
 
 #include <memory>
 #include <string>
 #include <vector>
 
@@ skipping 30 matching lines @@
 const char kMetricsNameAppInfoDelimiter = '#';
 
 std::unique_ptr<std::string> SerializeToJson(const base::Value& value) {
   std::unique_ptr<std::string> json_str(new std::string());
   JSONStringValueSerializer serializer(json_str.get());
   if (!serializer.Serialize(value))
     json_str.reset(nullptr);
   return json_str;
 }
 
+std::unique_ptr<base::DictionaryValue> CreateEventBase(
+    const std::string& name) {
+  std::unique_ptr<base::DictionaryValue> cast_event(
+      new base::DictionaryValue());
+  cast_event->SetString("name", name);
+  cast_event->SetDouble("time", base::TimeTicks::Now().ToInternalValue());
+
+  return cast_event;
+}
+
 }  // namespace
 
 // static
 
 // NOTE(gfhuang): This is a hacky way to encode/decode app infos into a
 // string. Mainly because it's hard to add another metrics serialization type
 // into components/metrics/serialization/.
 // static
 bool CastMetricsHelper::DecodeAppInfoFromMetricsName(
     const std::string& metrics_name,
@@ skipping 223 matching lines @@
 void CastMetricsHelper::LogMediumTimeHistogramEvent(
     const std::string& name,
     const base::TimeDelta& value) {
   // Follow UMA_HISTOGRAM_MEDIUM_TIMES definition.
   LogTimeHistogramEvent(name, value,
                         base::TimeDelta::FromMilliseconds(10),
                         base::TimeDelta::FromMinutes(3),
                         50);
 }
 
+void CastMetricsHelper::RecordEventWithValue(const std::string& event,
+                                             int value) {
+  std::unique_ptr<base::DictionaryValue> cast_event(CreateEventBase(event));
+  cast_event->SetInteger("value", value);
+  const std::string message = *SerializeToJson(*cast_event);
+  RecordSimpleAction(message);
+}
+
 void CastMetricsHelper::RecordApplicationEvent(const std::string& event) {
-  std::unique_ptr<base::DictionaryValue> cast_event(
-      new base::DictionaryValue());
-  cast_event->SetString("name", event);
-  base::TimeTicks now = base::TimeTicks::Now();
-  cast_event->SetDouble("time", now.ToInternalValue());
+  std::unique_ptr<base::DictionaryValue> cast_event(CreateEventBase(event));
   cast_event->SetString("app_id", app_id_);
   cast_event->SetString("session_id", session_id_);
   cast_event->SetString("sdk_version", sdk_version_);
-  const std::string message = *SerializeToJson(*cast_event.get()).get();
+  const std::string message = *SerializeToJson(*cast_event);
   RecordSimpleAction(message);
 }
 
 void CastMetricsHelper::RecordApplicationEventWithValue(
     const std::string& event,
     int value) {
-  std::unique_ptr<base::DictionaryValue> cast_event(
-      new base::DictionaryValue());
-  cast_event->SetString("name", event);
-  base::TimeTicks now = base::TimeTicks::Now();
-  cast_event->SetDouble("time", now.ToInternalValue());
+  std::unique_ptr<base::DictionaryValue> cast_event(CreateEventBase(event));
   cast_event->SetString("app_id", app_id_);
   cast_event->SetString("session_id", session_id_);
   cast_event->SetString("sdk_version", sdk_version_);
   cast_event->SetInteger("value", value);
-  const std::string message = *SerializeToJson(*cast_event.get()).get();
+  const std::string message = *SerializeToJson(*cast_event);
   RecordSimpleAction(message);
 }
 
 }  // namespace metrics
 }  // namespace chromecast