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

Unified Diff: remoting/proto/control.proto

Issue 8351084: Remove old Authentication code that we don't use or need. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: - Created 9 years, 1 month 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/proto/control.proto
diff --git a/remoting/proto/control.proto b/remoting/proto/control.proto
index 1f75bcb0500729fa1d45ed3305ec07efa0dc7dea..1d102edfaad196915e5c98be7d1ca04032a00d86 100644
--- a/remoting/proto/control.proto
+++ b/remoting/proto/control.proto
@@ -10,4 +10,16 @@ option optimize_for = LITE_RUNTIME;
package remoting.protocol;
-// Add control messages here. Currently we don't have any.
+// BeginSessionDepricated is send from host to client when session
Wez 2011/11/03 18:14:37 typo: Depricated -> Deprecated typo: send -> sent
Sergey Ulanov 2011/11/03 19:11:55 Done.
+// starts. Legacy clients expect to receive this message in the
Wez 2011/11/03 18:14:37 typo: in -> at
Sergey Ulanov 2011/11/03 19:11:55 Done.
+// beginning of each session so the host sends this message. Current
Wez 2011/11/03 18:14:37 nit: lose "so the host..."?
Sergey Ulanov 2011/11/03 19:11:55 Done.
+// client ignores it.
Wez 2011/11/03 18:14:37 typo: client -> clients, ignores -> ignore
Sergey Ulanov 2011/11/03 19:11:55 Done.
+//
+// TODO(sergeyu): Remove it once all clients are upgraded to the new
+// version (Chrome 17 is released).
Wez 2011/11/03 18:14:37 Release of M17 doesn't guarantee that all clients
Wez 2011/11/03 20:39:39 I think we may have to wait to remove this until w
+message LocalLoginStatusDeprecated {
+ optional bool success = 1;
+}
+message BeginSessionDeprecated {
+ optional LocalLoginStatusDeprecated login_status = 1;
+}

Powered by Google App Engine
This is Rietveld 408576698