TelemetryLogWriter cleanups

remoting/signaling/chromoting_event.h

Index: remoting/signaling/chromoting_event.h
diff --git a/remoting/signaling/chromoting_event.h b/remoting/signaling/chromoting_event.h
deleted file mode 100644
index 70bbb506809c6ac321c66bce63c819d6006ede7b..0000000000000000000000000000000000000000
--- a/remoting/signaling/chromoting_event.h
+++ /dev/null
@@ -1,151 +0,0 @@
-// Copyright 2016 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.
-
-#ifndef REMOTING_SIGNALING_CHROMOTING_EVENT_H_
-#define REMOTING_SIGNALING_CHROMOTING_EVENT_H_
-
-#include <memory>
-#include <string>
-
-#include "base/values.h"
-
-namespace remoting {
-
-// This is the representation of the log entry being sent to the telemetry
-// server. The content should be synced with chromoting_event.js and
-// chromoting_extensions.proto.
-class ChromotingEvent {
- public:
-  enum class ConnectionError {
-    NONE = 1,
-    HOST_OFFLINE = 2,
-    SESSION_REJECTED = 3,
-    INCOMPATIBLE_PROTOCOL = 4,
-    NETWORK_FAILURE = 5,
-    UNKNOWN_ERROR = 6,
-    INVALID_ACCESS_CODE = 7,
-    MISSING_PLUGIN = 8,
-    AUTHENTICATION_FAILED = 9,
-    BAD_VERSION = 10,
-    HOST_OVERLOAD = 11,
-    P2P_FAILURE = 12,
-    UNEXPECTED = 13,
-    CLIENT_SUSPENDED = 14,
-    NACL_DISABLED = 15,
-    MAX_SESSION_LENGTH = 16,
-    HOST_CONFIGURATION_ERROR = 17,
-    NACL_PLUGIN_CRASHED = 18
-  };
-
-  enum class Mode { IT2ME = 1, ME2ME = 2, LGAPP = 3 };
-
-  // macro defines from command line have polluted OS names like
-  // "LINUX", "ANDROID", etc.
-  enum class Os {
-    CHROMOTING_LINUX = 1,
-    CHROMOTING_CHROMEOS = 2,
-    CHROMOTING_MAC = 3,
-    CHROMOTING_WINDOWS = 4,
-    OTHER = 5,
-    CHROMOTING_ANDROID = 6,
-    CHROMOTING_IOS = 7
-  };
-
-  enum class Role { CLIENT = 0, HOST = 1 };
-
-  enum class SessionState {
-    UNKNOWN = 1,               // deprecated.
-    CREATED = 2,               // deprecated.
-    BAD_PLUGIN_VERSION = 3,    // deprecated.
-    UNKNOWN_PLUGIN_ERROR = 4,  // deprecated.
-    CONNECTING = 5,
-    INITIALIZING = 6,  // deprecated.
-    CONNECTED = 7,
-    CLOSED = 8,
-    CONNECTION_FAILED = 9,
-    UNDEFINED = 10,
-    PLUGIN_DISABLED = 11,  // deprecated.
-    CONNECTION_DROPPED = 12,
-    CONNECTION_CANCELED = 13,
-    AUTHENTICATED = 14,
-    STARTED = 15,
-    SIGNALING = 16,
-    CREATING_PLUGIN = 17,
-  };
-
-  enum class Type {
-    SESSION_STATE = 1,
-    CONNECTION_STATISTICS = 2,
-    SESSION_ID_OLD = 3,
-    SESSION_ID_NEW = 4,
-    HEARTBEAT = 5,
-    HEARTBEAT_REJECTED = 6,
-    RESTART = 7,
-    HOST_STATUS = 8,
-    SIGNAL_STRATEGY_PROGRESS = 9
-  };
-
-  static const char kCaptureLatencyKey[];
-  static const char kConnectionErrorKey[];
-  static const char kCpuKey[];
-  static const char kDecodeLatencyKey[];
-  static const char kEncodeLatencyKey[];
-  static const char kMaxCaptureLatencyKey[];
-  static const char kMaxDecodeLatencyKey[];
-  static const char kMaxEncodeLatencyKey[];
-  static const char kMaxRenderLatencyKey[];
-  static const char kMaxRoundtripLatencyKey[];
-  static const char kModeKey[];
-  static const char kOsKey[];
-  static const char kOsVersionKey[];
-  static const char kRenderLatencyKey[];
-  static const char kRoleKey[];
-  static const char kRoundtripLatencyKey[];
-  static const char kSessionDurationKey[];
-  static const char kSessionIdKey[];
-  static const char kSessionStateKey[];
-  static const char kTypeKey[];
-  static const char kVideoBandwidthKey[];
-
-  ChromotingEvent();
-  explicit ChromotingEvent(Type type);
-  ChromotingEvent(const ChromotingEvent& other);
-  ChromotingEvent(ChromotingEvent&& other);
-
-  ~ChromotingEvent();
-
-  ChromotingEvent& operator=(const ChromotingEvent& other);
-  ChromotingEvent& operator=(ChromotingEvent&& other);
-
-  // Sets an arbitrary key/value entry.
-  void SetString(const std::string& key, const std::string& value);
-  void SetInteger(const std::string& key, int value);
-  void SetBoolean(const std::string& key, bool value);
-  void SetDouble(const std::string& key, double value);
-  template <typename E>
-  void SetEnum(const std::string& key, E e) {
-    SetInteger(key, static_cast<int>(e));
-  }
-
-  // Adds fields of CPU type, OS type and OS version.
-  void AddSystemInfo();
-
-  void IncrementTryCount();
-  int try_count() const { return try_count_; }
-
-  // Returns a copy of the internal dictionary value.
-  std::unique_ptr<base::DictionaryValue> CopyDictionaryValue() const;
-
-  // Returns true if the SessionState concludes the end of session.
-  static bool IsEndOfSession(SessionState state);
-
- private:
-  std::unique_ptr<base::DictionaryValue> values_map_;
-
-  int try_count_ = 0;
-};
-
-}  // namespace remoting
-
-#endif  // REMOTING_SIGNALING_CHROMOTING_EVENT_H_