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

Side by Side Diff: remoting/protocol/client_stub.h

Issue 6724033: Remove authenticated_ fields from stubs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix .gyp file for unit tests. Created 9 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Interface of a client that receives commands from a Chromoting host. 5 // Interface of a client that receives commands from a Chromoting host.
6 // 6 //
7 // This interface is responsible for a subset of control messages sent to 7 // This interface is responsible for a subset of control messages sent to
8 // the Chromoting client. 8 // the Chromoting client.
9 9
10 #ifndef REMOTING_PROTOCOL_CLIENT_STUB_H_ 10 #ifndef REMOTING_PROTOCOL_CLIENT_STUB_H_
11 #define REMOTING_PROTOCOL_CLIENT_STUB_H_ 11 #define REMOTING_PROTOCOL_CLIENT_STUB_H_
12 12
13 #include "base/basictypes.h" 13 #include "base/basictypes.h"
14 14
15 class Task; 15 class Task;
16 16
17 namespace remoting { 17 namespace remoting {
18 namespace protocol { 18 namespace protocol {
19 19
20 class LocalLoginStatus; 20 class LocalLoginStatus;
21 class NotifyResolutionRequest; 21 class NotifyResolutionRequest;
22 22
23 class ClientStub { 23 class ClientStub {
24 public: 24 public:
25 ClientStub(); 25 ClientStub() {}
26 virtual ~ClientStub(); 26 virtual ~ClientStub() {}
27 27
28 virtual void NotifyResolution(const NotifyResolutionRequest* msg, 28 virtual void NotifyResolution(const NotifyResolutionRequest* msg,
29 Task* done) = 0; 29 Task* done) = 0;
30 virtual void BeginSessionResponse(const LocalLoginStatus* msg, 30 virtual void BeginSessionResponse(const LocalLoginStatus* msg,
31 Task* done) = 0; 31 Task* done) = 0;
32 32
33 // TODO(lambroslambrou): Remove OnAuthenticated() and OnClosed() when stubs
34 // are refactored not to store authentication state.
35
36 // Called when the client has authenticated with the host to enable the
37 // host->client control channel.
38 // Before this is called, only a limited set of control messages will be
39 // processed.
40 void OnAuthenticated();
41
42 // Called when the client is no longer connected.
43 void OnClosed();
44
45 // Has the client successfully authenticated with the host?
46 // I.e., should we be processing control events?
47 bool authenticated();
48
49 private: 33 private:
50 // Initially false, this records whether the client has authenticated with
51 // the host.
52 bool authenticated_;
53
54 DISALLOW_COPY_AND_ASSIGN(ClientStub); 34 DISALLOW_COPY_AND_ASSIGN(ClientStub);
55 }; 35 };
56 36
57 } // namespace protocol 37 } // namespace protocol
58 } // namespace remoting 38 } // namespace remoting
59 39
60 #endif // REMOTING_PROTOCOL_CLIENT_STUB_H_ 40 #endif // REMOTING_PROTOCOL_CLIENT_STUB_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698