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

Side by Side Diff: remoting/client/chromoting_client.h

Issue 7466004: Message validation in message dispatchers. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: - Created 9 years, 5 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 | « no previous file | remoting/client/chromoting_client.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 // ChromotingClient is the controller for the Client implementation. 5 // ChromotingClient is the controller for the Client implementation.
6 6
7 #ifndef REMOTING_CLIENT_CHROMOTING_CLIENT_H_ 7 #ifndef REMOTING_CLIENT_CHROMOTING_CLIENT_H_
8 #define REMOTING_CLIENT_CHROMOTING_CLIENT_H_ 8 #define REMOTING_CLIENT_CHROMOTING_CLIENT_H_
9 9
10 #include <list> 10 #include <list>
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
62 62
63 // Sets the viewport to do display. The viewport may be larger and/or 63 // Sets the viewport to do display. The viewport may be larger and/or
64 // smaller than the actual image background being displayed. 64 // smaller than the actual image background being displayed.
65 // 65 //
66 // TODO(ajwong): This doesn't make sense to have here. We're going to have 66 // TODO(ajwong): This doesn't make sense to have here. We're going to have
67 // threading isseus since pepper view needs to be called from the main pepper 67 // threading isseus since pepper view needs to be called from the main pepper
68 // thread synchronously really. 68 // thread synchronously really.
69 virtual void SetViewport(int x, int y, int width, int height); 69 virtual void SetViewport(int x, int y, int width, int height);
70 70
71 // ConnectionToHost::HostEventCallback implementation. 71 // ConnectionToHost::HostEventCallback implementation.
72 virtual void OnConnectionOpened(protocol::ConnectionToHost* conn); 72 virtual void OnConnectionOpened(protocol::ConnectionToHost* conn) OVERRIDE;
73 virtual void OnConnectionClosed(protocol::ConnectionToHost* conn); 73 virtual void OnConnectionClosed(protocol::ConnectionToHost* conn) OVERRIDE;
74 virtual void OnConnectionFailed(protocol::ConnectionToHost* conn); 74 virtual void OnConnectionFailed(protocol::ConnectionToHost* conn) OVERRIDE;
75 75
76 // ClientStub implementation. 76 // ClientStub implementation.
77 virtual void NotifyResolution(const protocol::NotifyResolutionRequest* msg,
78 Task* done);
79 virtual void BeginSessionResponse(const protocol::LocalLoginStatus* msg, 77 virtual void BeginSessionResponse(const protocol::LocalLoginStatus* msg,
80 Task* done); 78 Task* done) OVERRIDE;
81 79
82 // VideoStub implementation. 80 // VideoStub implementation.
83 virtual void ProcessVideoPacket(const VideoPacket* packet, Task* done); 81 virtual void ProcessVideoPacket(const VideoPacket* packet,
84 virtual int GetPendingPackets(); 82 Task* done) OVERRIDE;
83 virtual int GetPendingPackets() OVERRIDE;
85 84
86 private: 85 private:
87 struct QueuedVideoPacket { 86 struct QueuedVideoPacket {
88 QueuedVideoPacket(const VideoPacket* packet, Task* done) 87 QueuedVideoPacket(const VideoPacket* packet, Task* done)
89 : packet(packet), done(done) { 88 : packet(packet), done(done) {
90 } 89 }
91 const VideoPacket* packet; 90 const VideoPacket* packet;
92 Task* done; 91 Task* done;
93 }; 92 };
94 93
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
142 int64 last_sequence_number_; 141 int64 last_sequence_number_;
143 142
144 DISALLOW_COPY_AND_ASSIGN(ChromotingClient); 143 DISALLOW_COPY_AND_ASSIGN(ChromotingClient);
145 }; 144 };
146 145
147 } // namespace remoting 146 } // namespace remoting
148 147
149 DISABLE_RUNNABLE_METHOD_REFCOUNT(remoting::ChromotingClient); 148 DISABLE_RUNNABLE_METHOD_REFCOUNT(remoting::ChromotingClient);
150 149
151 #endif // REMOTING_CLIENT_CHROMOTING_CLIENT_H_ 150 #endif // REMOTING_CLIENT_CHROMOTING_CLIENT_H_
OLDNEW
« no previous file with comments | « no previous file | remoting/client/chromoting_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698