Pass MotionEvent tilt angles to Blink on Android.

ui/events/blink/blink_event_util.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // MSVC++ requires this to be set before any other includes to get M_PI.
 #define _USE_MATH_DEFINES
 
 #include "ui/events/blink/blink_event_util.h"
 
 #include <cmath>
@@ skipping 85 matching lines @@
   float major_radius = event.GetTouchMajor(pointer_index) / 2.f;
   float minor_radius = event.GetTouchMinor(pointer_index) / 2.f;
 
   DCHECK_LE(minor_radius, major_radius);
   DCHECK_IMPLIES(major_radius, minor_radius);
 
   float orientation_deg = event.GetOrientation(pointer_index) * 180.f / M_PI;
   DCHECK_GE(major_radius, 0);
   DCHECK_GE(minor_radius, 0);
   DCHECK_GE(major_radius, minor_radius);
   if (event.GetToolType(pointer_index) == MotionEvent::TOOL_TYPE_STYLUS) {

[mustaq, 2015/06/17 20:01:27]

We should convert MotionEvent orientation+tilt to WebTouchPoint tiltx+tilty
here, see my comment in crrev.com/1192563002.

[USE eero AT chromium.org, 2015/06/18 11:34:07]

Done.

     // Orientation lies in [-180, 180] for a stylus. Normalise to [-90, 90).
     // Allow a small bound tolerance to account for floating point conversion.
-    // TODO(e_hakkinen): crbug.com/493728: Pass also unaltered orientation
-    //                   to touch in order not to lose quadrant information.
     DCHECK_GT(orientation_deg, -180.01f);
     DCHECK_LT(orientation_deg, 180.01f);
     if (orientation_deg >= 90.f)
       orientation_deg -= 180.f;
     else if (orientation_deg < -90.f)
       orientation_deg += 180.f;
   } else {
     // Orientation lies in [-90, 90] for a touch. Normalise to [-90, 90).
     // Allow a small bound tolerance to account for floating point conversion.
     DCHECK_GT(orientation_deg, -90.01f);
     DCHECK_LT(orientation_deg, 90.01f);
     if (orientation_deg >= 90.f)
       orientation_deg -= 180.f;
   }
   if (orientation_deg >= 0) {
     // The case orientation_deg == 0 is handled here on purpose: although the
     // 'else' block is equivalent in this case, we want to pass the 0 value
     // unchanged (and 0 is the default value for many devices that don't
     // report elliptical touches).
     touch.radiusX = minor_radius;
     touch.radiusY = major_radius;
     touch.rotationAngle = orientation_deg;
   } else {
     touch.radiusX = major_radius;
     touch.radiusY = minor_radius;
     touch.rotationAngle = orientation_deg + 90;
   }
 
   touch.force = event.GetPressure(pointer_index);
+  // TODO(e_hakkinen): Leave only either tilt and tiltOrientation, tiltRad and
+  //                   tiltOrientationRad or tiltX and tiltY.
+  touch.tilt = event.GetTilt(pointer_index) * 180.f / M_PI;
+  touch.tiltRad = event.GetTilt(pointer_index);
+  touch.tiltOrientation =
+      event.GetTiltOrientation(pointer_index) * 180.f / M_PI;
+  touch.tiltOrientationRad = event.GetTiltOrientation(pointer_index);
+  touch.tiltX = event.GetTiltX(pointer_index) * 180.f / M_PI;
+  touch.tiltY = event.GetTiltY(pointer_index) * 180.f / M_PI;
 
   return touch;
 }
 
 }  // namespace
 
 blink::WebTouchEvent CreateWebTouchEventFromMotionEvent(
     const MotionEvent& event,
     bool may_cause_scrolling) {
   static_assert(static_cast<int>(MotionEvent::MAX_TOUCH_POINT_COUNT) ==
@@ skipping 160 matching lines @@
 }
 
 WebGestureEvent CreateWebGestureEventFromGestureEventData(
     const GestureEventData& data) {
   return CreateWebGestureEvent(data.details, data.time - base::TimeTicks(),
                                gfx::PointF(data.x, data.y),
                                gfx::PointF(data.raw_x, data.raw_y), data.flags);
 }
 
 }  // namespace ui