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

Unified Diff: remoting/protocol/client_stub.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_message_dispatcher.cc ('k') | remoting/protocol/client_stub.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/protocol/client_stub.h
diff --git a/remoting/protocol/client_stub.h b/remoting/protocol/client_stub.h
index 4b73789b84abeb95623275e5bee259665e2447b8..a2bd5653e56ea7272f2a63241968649544b71433 100644
--- a/remoting/protocol/client_stub.h
+++ b/remoting/protocol/client_stub.h
@@ -22,15 +22,29 @@ class NotifyResolutionRequest;
class ClientStub {
public:
- ClientStub() {}
- virtual ~ClientStub() {}
+ ClientStub();
+ virtual ~ClientStub();
virtual void NotifyResolution(const NotifyResolutionRequest* msg,
Task* done) = 0;
virtual void BeginSessionResponse(const LocalLoginStatus* msg,
Task* done) = 0;
+ // Called when the client has authenticated with the host to enable the
+ // host->client control channel.
+ // Before this is called, only a limited set of control messages will be
+ // processed.
+ void OnAuthenticated();
+
+ // Has the client successfully authenticated with the host?
+ // I.e., should we be processing control events?
+ bool authenticated();
+
private:
+ // Initially false, this records whether the client has authenticated with
+ // the host.
+ bool authenticated_;
+
DISALLOW_COPY_AND_ASSIGN(ClientStub);
};
« no previous file with comments | « remoting/protocol/client_message_dispatcher.cc ('k') | remoting/protocol/client_stub.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698