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

Unified Diff: remoting/host/it2me/it2me_host.h

Issue 2452223002: It2Me Host changes to better support Confirmation Dialog (Closed)
Patch Set: Fixing unit tests Created 4 years, 2 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 | « no previous file | remoting/host/it2me/it2me_host.cc » ('j') | remoting/host/it2me/it2me_host.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/it2me/it2me_host.h
diff --git a/remoting/host/it2me/it2me_host.h b/remoting/host/it2me/it2me_host.h
index 08a2bcedb71bed085e9d473c8a3ac49a1c08cf40..f2e8565ab09e5de6b85b4ff4e424f18b88676437 100644
--- a/remoting/host/it2me/it2me_host.h
+++ b/remoting/host/it2me/it2me_host.h
@@ -37,8 +37,8 @@ class PolicyWatcher;
class RegisterSupportHostRequest;
class RsaKeyPair;
-// These state values are duplicated in host_session.js and the Android Java
-// It2MeObserver class. Remember to update all copies when making changes.
+// These state values are duplicated in host_session.js. Remember to update
+// both copies when making changes.
enum It2MeHostState {
kDisconnected,
kStarting,
@@ -46,7 +46,8 @@ enum It2MeHostState {
kReceivedAccessCode,
kConnected,
kError,
- kInvalidDomainError
+ kInvalidDomainError,
+ kConnecting,
Sergey Ulanov 2016/10/27 22:44:12 I don't think there is any reason to keep the curr
joedow 2016/10/31 17:16:07 I wanted to make sure that mismatched host/clients
Sergey Ulanov 2016/10/31 18:21:17 The state is sent from host to client as a string,
joedow 2016/10/31 18:58:39 Ah, I didn't expect that we were using a stringifi
};
// Internal implementation of the plugin's It2Me host function.
« no previous file with comments | « no previous file | remoting/host/it2me/it2me_host.cc » ('j') | remoting/host/it2me/it2me_host.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698