Replacing WM_TOUCH with WM_POINTER for touch events on Wins 8+

ui/base/test/ui_controls_internal_win.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.
 
 #include "ui/base/test/ui_controls_internal_win.h"
 
 #include "base/bind.h"
 #include "base/callback.h"
 #include "base/location.h"
 #include "base/logging.h"
@@ skipping 312 matching lines @@
   input.mi.dwFlags = up_flags;
   if ((state & UP) && !::SendInput(1, &input, sizeof(INPUT)))
     return false;
 
   if (dispatcher.get())
     dispatcher->AddRef();
 
   return true;
 }
 
+bool SendTouchEventsImpl(int action,
+                         int num,
+                         long x,
+                         long y,
+                         const base::Closure& task) {
+  InitializeTouchInjection(num, TOUCH_FEEDBACK_INDIRECT);
+  scoped_refptr<InputDispatcher> dispatcher(
+      !task.is_null() ? new InputDispatcher(task, WM_POINTERDOWN) : NULL);
+
+  POINTER_TOUCH_INFO contact_[3];

[sky, 2017/06/07 19:42:45]

Where does the 3 come from?

[sky, 2017/06/07 19:42:45]

trailing '_' is only for members.

[lanwei, 2017/06/22 18:38:45]

I removed it.

[lanwei, 2017/06/22 18:38:45]

Done.

+  for (int i = 0; i < num; i++) {

[sky, 2017/06/07 19:42:45]

It's not at all clear to me what this code is doing, why you have 3 and num.
Please add comments.

+    POINTER_TOUCH_INFO& contact = contact_[i];
+    memset(&contact, 0, sizeof(POINTER_TOUCH_INFO));
+    contact.pointerInfo.pointerType = PT_TOUCH;
+    contact.pointerInfo.pointerId = i;
+    contact.pointerInfo.ptPixelLocation.y = y;
+    contact.pointerInfo.ptPixelLocation.x = x + 50 * i;
+
+    contact.touchFlags = TOUCH_FLAG_NONE;
+    contact.touchMask =
+        TOUCH_MASK_CONTACTAREA | TOUCH_MASK_ORIENTATION | TOUCH_MASK_PRESSURE;
+    contact.orientation = 90;
+    contact.pressure = 32000;
+
+    // defining contact area
+    contact.rcContact.top = contact.pointerInfo.ptPixelLocation.y - 2;
+    contact.rcContact.bottom = contact.pointerInfo.ptPixelLocation.y + 2;
+    contact.rcContact.left = contact.pointerInfo.ptPixelLocation.x - 2;
+    contact.rcContact.right = contact.pointerInfo.ptPixelLocation.x + 2;
+
+    contact.pointerInfo.pointerFlags =
+        POINTER_FLAG_DOWN | POINTER_FLAG_INRANGE | POINTER_FLAG_INCONTACT;
+  }
+  bool result =
+      InjectTouchInput(num, contact_);  // Injecting the touch down on screen

[ananta, 2017/06/09 03:13:12]

The reason the tests are failing on Windows 7 is a direct call to this function
which exists on Windows 8+. A GetProcAddress based approach would work here.
Alternatively delay load user32 may work. Dunno if that is feasible for this
case.

[lanwei, 2017/06/22 18:38:45]

Done.

+  Sleep(20);

[sky, 2017/06/07 19:42:45]

Why do you sleep between injecting events?

[lanwei, 2017/06/22 18:38:45]

Because the test is flaky, I want to make sure we have enough time to receive
the events from InjectTouchInput.

[sky, 2017/06/22 20:54:21]

The problem with random sleeps is that they generally mean something else is
wrong. To add random sleeps generally means your test is going to be flakey.
Remember that the bots have varying levels of load.

[lanwei, 2017/06/23 20:06:05]

I removed them.

+
+  if (action & MOVE) {
+    for (int i = 0; i < num; i++) {
+      POINTER_TOUCH_INFO& contact = contact_[i];
+      contact.pointerInfo.ptPixelLocation.y = y + 10;
+      contact.pointerInfo.ptPixelLocation.x = x + 50 * i + 30;
+      contact.pointerInfo.pointerFlags =
+          POINTER_FLAG_UPDATE | POINTER_FLAG_INRANGE | POINTER_FLAG_INCONTACT;
+    }
+    result =
+        result &&
+        InjectTouchInput(num, contact_);  // Injecting the touch move on screen
+    Sleep(20);
+  }
+
+  if (action & RELEASE) {
+    for (int i = 0; i < num; i++) {
+      POINTER_TOUCH_INFO& contact = contact_[i];
+      contact.pointerInfo.ptPixelLocation.y = y + 10;
+      contact.pointerInfo.ptPixelLocation.x = x + 50 * i + 30;
+      contact.pointerInfo.pointerFlags = POINTER_FLAG_UP | POINTER_FLAG_INRANGE;
+    }
+    result = result && InjectTouchInput(
+                           num, contact_);  // Injecting the touch up on screen
+    Sleep(20);
+  }
+
+  if (!result)
+    return false;
+
+  if (dispatcher.get())
+    dispatcher->AddRef();
+
+  return true;
+}
+
 }  // namespace internal
 }  // namespace ui_controls