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

Unified Diff: remoting/webapp/crd/js/client_session.js

Issue 1032553008: [Chromoting] Update client connection enums in JS to match C++. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 9 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
Index: remoting/webapp/crd/js/client_session.js
diff --git a/remoting/webapp/crd/js/client_session.js b/remoting/webapp/crd/js/client_session.js
index 0bb20cb674c12accc9598674044652370a83e10d..135c4901fdf177bbfd2bfdde54a7c8efc8ab4d37 100644
--- a/remoting/webapp/crd/js/client_session.js
+++ b/remoting/webapp/crd/js/client_session.js
@@ -98,20 +98,23 @@ remoting.ClientSession.Events = {
};
// Note that the positive values in both of these enums are copied directly
-// from chromoting_scriptable_object.h and must be kept in sync. The negative
-// values represent state transitions that occur within the web-app that have
-// no corresponding plugin state transition.
+// from connection_to_host.h and must be kept in sync. Code in
+// chromoting_instance.cc converts the C++ enums into strings that must match
+// the names given here.
+// The negative values represent state transitions that occur within the
+// web-app that have no corresponding plugin state transition.
/** @enum {number} */
remoting.ClientSession.State = {
CONNECTION_CANCELED: -3, // Connection closed (gracefully) before connecting.
CONNECTION_DROPPED: -2, // Succeeded, but subsequently closed with an error.
CREATED: -1,
UNKNOWN: 0,
- CONNECTING: 1,
- INITIALIZING: 2,
- CONNECTED: 3,
- CLOSED: 4,
- FAILED: 5
+ INITIALIZING: 1,
garykac 2015/03/24 20:09:13 Match order in connection_to_host
+ CONNECTING: 2,
+ AUTHENTICATED: 3,
garykac 2015/03/24 20:09:13 Add missing AUTHENTICATED
+ CONNECTED: 4,
+ CLOSED: 5,
+ FAILED: 6
};
/**
« remoting/protocol/connection_to_host.h ('K') | « remoting/protocol/connection_to_host.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698