client-side wheel mouse support

remoting/proto/event.proto

Index: remoting/proto/event.proto
diff --git a/remoting/proto/event.proto b/remoting/proto/event.proto
index 945d0b0e27175ba911d5ddc44608dbf52bae4d26..d575eb8582fa5371d284ad3aca53c824344fbc29 100644
--- a/remoting/proto/event.proto
+++ b/remoting/proto/event.proto
@@ -11,7 +11,6 @@ option optimize_for = LITE_RUNTIME;
 package remoting.protocol;
 
 // Defines a keyboard event.
-// NEXT ID: 3
 message KeyEvent {
   // The POSIX key code.
   required int32 keycode = 1;
@@ -33,8 +32,16 @@ message MouseEvent {
   optional int32 y = 2;
 
   // Mouse wheel information.
+  // In general, the |offset| values are the most useful, but the raw |clicks|
+  // value can be useful when cycling through a small set of items.

[Jamie, 2011/03/16 11:59:29]

I don't understand what this comment means. How would a host decide which of the
values to use?

[garykac, 2011/03/16 20:43:43]

If you're doing scrolling, then you should use the offsets.

If you're not doing traditional scrolling, then you can consider using the
clicks instead. An example of when you might want to do this would be when
cycling through the set of tabs in Chrome. This isn't really scrolling, but you
can use the wheel clicks to select the previous/next tabs.

+  // The number of pixels to scroll.
   optional int32 wheel_offset_x = 3;
   optional int32 wheel_offset_y = 4;
+  // The number of wheel clicks.
+  optional int32 wheel_clicks_x = 7;
+  optional int32 wheel_clicks_y = 8;
+  // True if |offset| values mean scroll by page (rather than by pixel or line).

[Jamie, 2011/03/16 11:59:29]

Why do we have a bool to indicate scrolling by a page, but not one to
distinguish between pixels vs. lines if wheel_by_page is false?

[garykac, 2011/03/16 20:43:43]

The pixel vs. line distinction is app-specific.

I understand this as:

If false, then scroll by the smallest unit of scrolliness that you (the app)
understand. This is most commonly either pixels or lines.

If true, then scroll by an entire page of content.

+  optional bool wheel_by_page = 9;
 
   // Mouse button event.
   optional MouseButton button = 5;