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

Unified Diff: remoting/proto/event.proto

Issue 6271004: Changed MessageReader so that it doesn't read from the socket if there are (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: proper handling of empty messages Created 9 years, 11 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
Index: remoting/proto/event.proto
diff --git a/remoting/proto/event.proto b/remoting/proto/event.proto
index 19c734da97184e59818e2181faca4099d4296de2..10a6b8291606133b4cb324adef0bd530d9ea8cfe 100644
--- a/remoting/proto/event.proto
+++ b/remoting/proto/event.proto
@@ -42,15 +42,10 @@ message MouseEvent {
}
// Defines an event message on the event channel.
-message Event {
+message EventMessage {
awong 2011/01/20 20:06:38 Nit, I would just called his "event" or better "In
Sergey Ulanov 2011/01/20 21:55:57 I think it is better to call it EventMessage for c
required int32 timestamp = 1; // Client timestamp for event
optional bool dummy = 2; // Is this a dummy event?
optional KeyEvent key = 3;
optional MouseEvent mouse = 4;
}
-
-// Message sent in the event channel.
-message EventMessage {
- repeated Event event = 1;
-}
« no previous file with comments | « no previous file | remoting/protocol/client_message_dispatcher.h » ('j') | remoting/protocol/client_message_dispatcher.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698