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

Unified Diff: remoting/protocol/connection_to_client.h

Issue 6594138: Block event processing on host/client until the client has authenticated. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix merge conflict Created 9 years, 10 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/protocol/client_stub.cc ('k') | remoting/protocol/connection_to_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/connection_to_client.h
diff --git a/remoting/protocol/connection_to_client.h b/remoting/protocol/connection_to_client.h
index b7aa052c9a298282f093bb4eb8a9b48e39bcf9b7..d2721f3303c2c34689882b27128e487502b55685 100644
--- a/remoting/protocol/connection_to_client.h
+++ b/remoting/protocol/connection_to_client.h
@@ -72,9 +72,8 @@ class ConnectionToClient :
// Return pointer to ClientStub.
virtual ClientStub* client_stub();
- protected:
- // Protected constructor used by unit test.
- ConnectionToClient();
+ // Called when the host accepts the client authentication.
+ void OnClientAuthenticated();
private:
// Callback for protocol Session.
@@ -85,6 +84,11 @@ class ConnectionToClient :
void OnClosed();
+ // Initially false, this is set to true once the client has authenticated
+ // properly. When this is false, many client messages (like input events)
+ // will be ignored.
+ bool client_authenticated_;
+
// The libjingle channel used to send and receive data from the remote client.
scoped_refptr<Session> session_;
« no previous file with comments | « remoting/protocol/client_stub.cc ('k') | remoting/protocol/connection_to_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698