[Remoting Android] Refactor ClientTelemetryLogger

remoting/client/jni/jni_client.h

Index: remoting/client/jni/jni_client.h
diff --git a/remoting/client/jni/jni_client.h b/remoting/client/jni/jni_client.h
index 412d2e8ba7db68ebe70bdcd305796e43fd1aeaa7..0f38b4a730268bf66775ef2ee917070804e5d695 100644
--- a/remoting/client/jni/jni_client.h
+++ b/remoting/client/jni/jni_client.h
@@ -10,6 +10,7 @@
 #include "base/android/scoped_java_ref.h"
 #include "base/macros.h"
 #include "base/memory/weak_ptr.h"
+#include "remoting/client/jni/connect_to_host_info.h"

[Sergey Ulanov, 2017/01/19 00:50:39]

ConnectToHostInfo can be forward-declared.
https://chromium.googlesource.com/chromium/src/+/master/styleguide/c++/c++.md...

[Yuwei, 2017/01/19 23:00:00]

Done.

 #include "remoting/protocol/connection_to_host.h"
 #include "remoting/protocol/cursor_shape_stub.h"
 
@@ -33,18 +34,7 @@ class JniClient {
   // Initiates a connection with the specified host. To skip the attempt at
   // pair-based authentication, leave |pairing_id| and |pairing_secret| as
   // empty strings.
-  void ConnectToHost(const std::string& username,
-                     const std::string& auth_token,
-                     const std::string& host_jid,
-                     const std::string& host_id,
-                     const std::string& host_pubkey,
-                     const std::string& pairing_id,
-                     const std::string& pairing_secret,
-                     const std::string& capabilities,
-                     const std::string& flags,
-                     const std::string& host_version,
-                     const std::string& host_os,
-                     const std::string& host_os_version);
+  void ConnectToHost(const ConnectToHostInfo& info);
 
   // Terminates any ongoing connection attempt and cleans up by nullifying
   // |session_|. This is a no-op unless |session| is currently non-null.