Correct missing radius in ui::TouchEvent.

content/browser/renderer_host/input/web_input_event_util.cc

 // Copyright 2013 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 "content/browser/renderer_host/input/web_input_event_util.h"
 
 #include <cmath>
@@ skipping 192 matching lines @@
   // radians. See:
   // http://developer.android.com/reference/android/view/MotionEvent.html
   //
   // The proposed extension to W3C Touch Events specifies the touch ellipse
   // using two radii along x- & y-axes and a positive acute rotation angle in
   // degrees. See:
   // http://dvcs.w3.org/hg/webevents/raw-file/default/touchevents.html
 
   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_GT(major_radius, 0);

[jdduke (slow), 2015/01/13 21:08:28]

I'm not sure we can assume that major_radius is non-zero though.

+
   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);
   // 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 >= 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
@@ skipping 230 matching lines @@
     flags |= ui::EF_CAPS_LOCK_DOWN;
   if (modifiers & blink::WebInputEvent::IsAutoRepeat)
     flags |= ui::EF_IS_REPEAT;
   if (modifiers & blink::WebInputEvent::IsKeyPad)
     flags |= ui::EF_NUMPAD_KEY;
 
   return flags;
 }
 
 }  // namespace content