Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(87)

Unified Diff: remoting/client/ios/bridge/client_instance.h

Issue 2753963002: Refactoring and rewriting the chromoting jni instance to be chromoting session. (Closed)
Patch Set: Updating based on feedback. Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/client/connect_to_host_info.cc ('k') | remoting/client/ios/bridge/client_instance.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/client/ios/bridge/client_instance.h
diff --git a/remoting/client/ios/bridge/client_instance.h b/remoting/client/ios/bridge/client_instance.h
index 1ccaccf620dee570e22f980219d9c5dcb380f40e..e769f412f3b80d6ddab7d206dad13213a80cd89c 100644
--- a/remoting/client/ios/bridge/client_instance.h
+++ b/remoting/client/ios/bridge/client_instance.h
@@ -36,11 +36,12 @@ namespace remoting {
// class AudioPlayerIos;
class ClientProxy;
-class ClientStatusLogger;
+// class ClientStatusLogger;
class FrameConsumerBridge;
-// ClientUserInterface that indirectly makes and receives OBJ_C calls from the
-// UI application.
+// TODO(nicholss): Delete this Class. It needs to use the client version of
+// ClientInstance. ClientUserInterface that indirectly makes and receives OBJ_C
+// calls from the UI application.
class ClientInstance : public ClientUserInterface,
public protocol::ClipboardStub,
public protocol::CursorShapeStub,
@@ -158,7 +159,9 @@ class ClientInstance : public ClientUserInterface,
std::unique_ptr<ChromotingClient> client_;
XmppSignalStrategy::XmppServerConfig xmpp_config_;
std::unique_ptr<XmppSignalStrategy> signaling_; // Must outlive client_
- std::unique_ptr<ClientStatusLogger> client_status_logger_;
+
+ // TODO(nicholss): Migrate to ClientTelemetryLogger
+ // std::unique_ptr<ClientStatusLogger> client_status_logger_;
// This group of variables is to be used on the audio thread.
// std::unique_ptr<AudioPlayerIos> audio_player_;
« no previous file with comments | « remoting/client/connect_to_host_info.cc ('k') | remoting/client/ios/bridge/client_instance.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698