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

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

Issue 6724033: Remove authenticated_ fields from stubs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address reviewer's comments. Created 9 years, 8 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
« no previous file with comments | « remoting/protocol/host_message_dispatcher.cc ('k') | remoting/protocol/host_stub.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2011 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 host that receives commands from a Chromoting client. 5 // Interface of a host that receives commands from a Chromoting client.
6 // 6 //
7 // This interface handles control messages defined in contro.proto. 7 // This interface handles control messages defined in control.proto.
8 8
9 #ifndef REMOTING_PROTOCOL_HOST_STUB_H_ 9 #ifndef REMOTING_PROTOCOL_HOST_STUB_H_
10 #define REMOTING_PROTOCOL_HOST_STUB_H_ 10 #define REMOTING_PROTOCOL_HOST_STUB_H_
11 11
12 #include "base/basictypes.h" 12 #include "base/basictypes.h"
13 13
14 class Task; 14 class Task;
15 15
16 namespace remoting { 16 namespace remoting {
17 namespace protocol { 17 namespace protocol {
18 18
19 class LocalLoginCredentials; 19 class LocalLoginCredentials;
20 class SuggestResolutionRequest; 20 class SuggestResolutionRequest;
21 21
22 class HostStub { 22 class HostStub {
23 public: 23 public:
24 HostStub(); 24 HostStub() {};
25 virtual ~HostStub(); 25 virtual ~HostStub() {};
26 26
27 virtual void SuggestResolution( 27 virtual void SuggestResolution(
28 const SuggestResolutionRequest* msg, Task* done) = 0; 28 const SuggestResolutionRequest* msg, Task* done) = 0;
29 virtual void BeginSessionRequest( 29 virtual void BeginSessionRequest(
30 const LocalLoginCredentials* credentials, Task* done) = 0; 30 const LocalLoginCredentials* credentials, Task* done) = 0;
31 31
32 // TODO(lambroslambrou): Remove OnAuthenticated() and OnClosed() when stubs
33 // are refactored not to store authentication state.
34
35 // Called when the client has authenticated with the host to enable the
36 // client->host control channel.
37 // Before this is called, only a limited set of control messages will be
38 // processed.
39 void OnAuthenticated();
40
41 // Called when the client is no longer connected.
42 void OnClosed();
43
44 // Has the client successfully authenticated with the host?
45 // I.e., should we be processing control events?
46 bool authenticated();
47
48 private: 32 private:
49 // Initially false, this records whether the client has authenticated with
50 // the host.
51 bool authenticated_;
52
53 DISALLOW_COPY_AND_ASSIGN(HostStub); 33 DISALLOW_COPY_AND_ASSIGN(HostStub);
54 }; 34 };
55 35
56 } // namespace protocol 36 } // namespace protocol
57 } // namespace remoting 37 } // namespace remoting
58 38
59 #endif // REMOTING_PROTOCOL_HOST_STUB_H_ 39 #endif // REMOTING_PROTOCOL_HOST_STUB_H_
OLDNEW
« no previous file with comments | « remoting/protocol/host_message_dispatcher.cc ('k') | remoting/protocol/host_stub.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698