Update rich touch information to conform to standard

views/events/event_x.cc

Index: views/events/event_x.cc
diff --git a/views/events/event_x.cc b/views/events/event_x.cc
index 3721fb26b6a67fc31eaca464fd79d06032d74585..2c7d65fa498263c6c4e31077e9529bbbde2f0320 100644
--- a/views/events/event_x.cc
+++ b/views/events/event_x.cc
@@ -245,13 +245,12 @@ uint16 GetCharacterFromXKeyEvent(XKeyEvent* key) {
           result.length() == 1) ? result[0] : 0;
 }
 
-float GetTouchRadiusFromXEvent(XEvent* xev) {
-  float diameter = 0.0;
+float GetTouchRadiusFromXEvent(XEvent* xev, TouchFactory::TouchParam tp) {
+  float diameter = 2.0;
 
 #if defined(HAVE_XINPUT2)
   TouchFactory* touch_factory = TouchFactory::GetInstance();
-  touch_factory->ExtractTouchParam(*xev, TouchFactory::TP_TOUCH_MAJOR,
-                                   &diameter);
+  touch_factory->ExtractTouchParam(*xev, tp, &diameter);
 #endif

[sadrul, 2011/06/08 17:30:14]

Let's have:

float GetTouchParamFromXEvent(XEvent* xev,
    TouchParam tp,
    float default_value) {
  float value = default_value;
#if defined(HAVE_XINPUT2)
  TouchFactory::GetInstance()->ExtractTouchParam(*xev, tp, &value);
#endif
  return value;
}

and use it in the constructor (doing ' / 2.0' for radius_x_ and radius_y_) and
remove GetTouchAngleFromXEvent too

[Yufeng Shen (Slow to review), 2011/06/08 19:39:24]

Done.

 
   return diameter / 2.0;
@@ -269,31 +268,6 @@ float GetTouchAngleFromXEvent(XEvent* xev) {
   return angle;
 }
 
-
-float GetTouchRatioFromXEvent(XEvent* xev) {
-  float ratio = 1.0;
-
-#if defined(HAVE_XINPUT2)
-  TouchFactory* touch_factory = TouchFactory::GetInstance();
-  float major_v = -1.0;
-  float minor_v = -1.0;
-
-  if (!touch_factory->ExtractTouchParam(*xev,
-                                        TouchFactory::TP_TOUCH_MAJOR,
-                                        &major_v) ||
-      !touch_factory->ExtractTouchParam(*xev,
-                                        TouchFactory::TP_TOUCH_MINOR,
-                                        &minor_v))
-    return ratio;
-
-  // In case minor axis exists but is zero.
-  if (minor_v > 0.0)
-    ratio = major_v / minor_v;
-#endif
-
-  return ratio;
-}
-
 }  // namespace
 
 ////////////////////////////////////////////////////////////////////////////////
@@ -421,9 +395,11 @@ TouchEvent::TouchEvent(NativeEvent2 native_event_2,
                        FromNativeEvent2 from_native)
     : LocatedEvent(native_event_2, from_native),
       touch_id_(GetTouchIDFromXEvent(native_event_2)),
-      radius_(GetTouchRadiusFromXEvent(native_event_2)),
-      angle_(GetTouchAngleFromXEvent(native_event_2)),
-      ratio_(GetTouchRatioFromXEvent(native_event_2)) {
+      radiusX_(GetTouchRadiusFromXEvent(native_event_2,
+                                        TouchFactory::TP_TOUCH_MAJOR)),
+      radiusY_(GetTouchRadiusFromXEvent(native_event_2,
+                                        TouchFactory::TP_TOUCH_MINOR)),
+      angle_(GetTouchAngleFromXEvent(native_event_2)) {
   if (type() == ui::ET_TOUCH_PRESSED || type() == ui::ET_TOUCH_RELEASED) {
     TouchFactory* factory = TouchFactory::GetInstance();
     float slot;