Chromium Code Reviews| Index: ui/events/gestures/motion_event_aura.cc |
| diff --git a/ui/events/gestures/motion_event_aura.cc b/ui/events/gestures/motion_event_aura.cc |
| index 1f4d2855b2fc0cfc5d3cd32495f5586d6f6cea13..2317ec7b4ffb0bd9d24012efe2369868e416d1f9 100644 |
| --- a/ui/events/gestures/motion_event_aura.cc |
| +++ b/ui/events/gestures/motion_event_aura.cc |
| @@ -28,11 +28,19 @@ PointerProperties GetPointerPropertiesFromTouchEvent(const TouchEvent& touch) { |
| float radius_x = touch.radius_x(); |
| float radius_y = touch.radius_y(); |
| float rotation_angle_rad = touch.rotation_angle() * M_PI / 180.f; |
| + |
| DCHECK_GE(radius_x, 0) << "Unexpected x-radius < 0"; |
| DCHECK_GE(radius_y, 0) << "Unexpected y-radius < 0"; |
| - DCHECK(0 <= rotation_angle_rad && rotation_angle_rad <= M_PI_2) |
| + DCHECK(0 <= rotation_angle_rad && rotation_angle_rad < M_PI) |
| << "Unexpected touch rotation angle"; |
| + // Make the angle acute to ease subsequent logic. The angle range effectively |
| + // changes from [0, pi) to [0, pi/2). |
| + if (rotation_angle_rad >= M_PI_2) { |
| + rotation_angle_rad -= (float) M_PI_2; |
|
tdresser
2015/01/21 17:09:08
Use static_cast<float>
mustaq
2015/01/21 18:31:55
Done.
|
| + std::swap(radius_x, radius_y); |
| + } |
| + |
| if (radius_x > radius_y) { |
| // The case radius_x == radius_y is omitted from here on purpose: for |
| // circles, we want to pass the angle (which could be any value in such |