Overhaul Mandoline event transport code

components/html_viewer/touch_handler.cc

Index: components/html_viewer/touch_handler.cc
diff --git a/components/html_viewer/touch_handler.cc b/components/html_viewer/touch_handler.cc
index 4764867eb15e7dc7be50c8f2100b61d494d0311d..3d7d96bf3cfedc533b13772827a4a8aed50fb324 100644
--- a/components/html_viewer/touch_handler.cc
+++ b/components/html_viewer/touch_handler.cc
@@ -15,6 +15,8 @@
 namespace html_viewer {
 namespace {
 
+// TODO(rjkroege): Gesture recognition currently happens in the html_viewer.
+// In phase2, it will be relocated to MUS. Update this code at that time.
 void SetPropertiesFromEvent(const mojo::Event& event,
                             ui::PointerProperties* properties) {
   properties->id = event.pointer_data->pointer_id;
@@ -22,10 +24,20 @@ void SetPropertiesFromEvent(const mojo::Event& event,
   properties->y = event.pointer_data->y;
   properties->raw_x = event.pointer_data->screen_x;
   properties->raw_y = event.pointer_data->screen_y;
-  properties->pressure = event.pointer_data->pressure;
-  properties->SetAxesAndOrientation(event.pointer_data->radius_major,
-                                    event.pointer_data->radius_minor,
-                                    event.pointer_data->orientation);
+
+  if (event.pointer_data->kind == mojo::POINTER_KIND_TOUCH ||
+      event.pointer_data->kind == mojo::POINTER_KIND_PEN) {
+    properties->pressure = event.pointer_data->brush_data->pressure;
+
+    // TODO(rjkroege): vary orientation for width, height.
+    properties->SetAxesAndOrientation(event.pointer_data->brush_data->width,
+                                      event.pointer_data->brush_data->height,
+                                      0.0);
+  } else {
+    // TODO(rjkroege): This might need to .5 when the button is down to
+    // satisfy the Pointer events specification.
+    properties->pressure = 0.0;

[sadrul, 2015/09/10 17:39:15]

We can look at (event.flags | EVENT_FLAGS_LEFT_MOUSE_BUTTON) here to decide
whether to set to 0.5 or not.

[rjkroege, 2015/09/12 01:29:02]

Done.

+  }
   // TODO(sky): Add support for tool_type.
 }