Android mouse events shouldn't appear as TouchEvents

content/browser/android/content_view_core_impl.h

 // Copyright 2012 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.
 
 #ifndef CONTENT_BROWSER_ANDROID_CONTENT_VIEW_CORE_IMPL_H_
 #define CONTENT_BROWSER_ANDROID_CONTENT_VIEW_CORE_IMPL_H_
 
 #include <stdint.h>
 
 #include <memory>
@@ skipping 110 matching lines @@
       jfloat orientation_1,
       jfloat tilt_0,
       jfloat tilt_1,
       jfloat raw_pos_x,
       jfloat raw_pos_y,
       jint android_tool_type_0,
       jint android_tool_type_1,
       jint android_button_state,
       jint android_meta_state,
       jboolean is_touch_handle_event);
-  jboolean SendMouseMoveEvent(JNIEnv* env,
-                              const base::android::JavaParamRef<jobject>& obj,
-                              jlong time_ms,
-                              jfloat x,
-                              jfloat y,
-                              jint tool_type);
+  jboolean SendMouseEvent(JNIEnv* env,
+                          const base::android::JavaParamRef<jobject>& obj,
+                          jlong time_ms,
+                          jint android_action,
+                          jfloat x,
+                          jfloat y,
+                          jint pointer_id,
+                          jfloat pressure,
+                          jfloat orientation,
+                          jfloat tilt,
+                          jint android_button_state,
+                          jint android_meta_state,
+                          jint tool_type);
   jboolean SendMouseWheelEvent(JNIEnv* env,
                                const base::android::JavaParamRef<jobject>& obj,
                                jlong time_ms,
                                jfloat x,
                                jfloat y,
                                jfloat ticks_x,
                                jfloat ticks_y,
                                jfloat pixels_per_tick);
   void ScrollBegin(JNIEnv* env,
                    const base::android::JavaParamRef<jobject>& obj,
@@ skipping 318 matching lines @@
 
   // Observer to notify of lifecyle changes.
   base::ObserverList<ContentViewCoreImplObserver> observer_list_;
 
   // The cache of device's current orientation set from Java side, this value
   // will be sent to Renderer once it is ready.
   int device_orientation_;
 
   bool accessibility_enabled_;
 
+  // The buttons currently depressed, see MotionEvent.getButtonState().
+  int button_state_;

[aelias_OOO_until_Jul13, 2016/10/20 23:26:29]

FWIW, this state doesn't really belong here, although I understand it's needed
to work around a bug...  I'm mulling over if there's any way it could be
avoided.  Could you rebase this patch so that I can apply it locally and observe
the buggy early-pressed MouseMoves?

[mustaq, 2016/10/21 19:19:08]

The latest patch is after a rebase. I have also added enter/leave plumbing
there, checked only on M. Still couldn't locate an N device here :(

[mustaq, 2016/10/26 17:45:09]

Discovered today that my last patch was unusable for unhandled hover-moves.
Fixed.

+
   // Manages injecting Java objects.
   scoped_refptr<GinJavaBridgeDispatcherHost> java_bridge_dispatcher_host_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentViewCoreImpl);
 };
 
 bool RegisterContentViewCore(JNIEnv* env);
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_ANDROID_CONTENT_VIEW_CORE_IMPL_H_