Pass TouchMajor to HitTestResult

android_webview/java/src/org/chromium/android_webview/AwContents.java

Index: android_webview/java/src/org/chromium/android_webview/AwContents.java
diff --git a/android_webview/java/src/org/chromium/android_webview/AwContents.java b/android_webview/java/src/org/chromium/android_webview/AwContents.java
index cb996fa2002677f431d198bf7989e9a65589d556..1827952ebfd079f2416cc1e19467e200b3129451 100644
--- a/android_webview/java/src/org/chromium/android_webview/AwContents.java
+++ b/android_webview/java/src/org/chromium/android_webview/AwContents.java
@@ -2347,11 +2347,16 @@ public class AwContents {
             if (event.getActionMasked() == MotionEvent.ACTION_DOWN) {
                 int actionIndex = event.getActionIndex();
 
+                // The length of major/minor axis of the touch area ellipse.
+                float touchMajor = event.getToolMajor(actionIndex) / (float) mDIPScale;
+                float touchMinor = event.getToolMinor(actionIndex) / (float) mDIPScale;

[jdduke (slow), 2014/10/30 19:54:58]

I wouldn't use touchMinor here, as then you'd also need to use the touch
orientation. Major/minor don't necessarily correspond to X/Y axes, so it should
be sufficient to just use touchMajor.

[hush (inactive), 2014/11/04 01:55:57]

Okay. In this case,  I am basically treating the touch area as a square, whose
width/height is touchMajor.

                 // Note this will trigger IPC back to browser even if nothing is
                 // hit.
                 nativeRequestNewHitTestDataAt(mNativeAwContents,
                         (int) Math.round(event.getX(actionIndex) / mDIPScale),
-                        (int) Math.round(event.getY(actionIndex) / mDIPScale));
+                        (int) Math.round(event.getY(actionIndex) / mDIPScale),
+                        touchMajor,
+                        touchMinor);
             }
 
             if (mOverScrollGlow != null) {
@@ -2565,7 +2570,8 @@ public class AwContents {
     private native byte[] nativeGetCertificate(long nativeAwContents);
 
     // Coordinates in desity independent pixels.
-    private native void nativeRequestNewHitTestDataAt(long nativeAwContents, int x, int y);
+    private native void nativeRequestNewHitTestDataAt(long nativeAwContents, int x, int y,
+            float touchMajor, float touchMinor);
     private native void nativeUpdateLastHitTestData(long nativeAwContents);
 
     private native void nativeOnSizeChanged(long nativeAwContents, int w, int h, int ow, int oh);