Open Bug 1495580 Opened 6 years ago Updated 3 months ago

Enable test_group_zoom.html on Windows (InjectTouchInput failure)


(Core :: Panning and Zooming, enhancement, P5)




Tracking Status
firefox64 --- affected


(Reporter: botond, Unassigned)


(Blocks 1 open bug)


In bug 1470504 we are enabling gfx/layers/apz/test/mochitest/test_group_zoom.html on desktop platforms except for Windows.

This bug tracks enabling it for Windows as well.
Priority: P3 → P5

A new test I added in bug 1636911


fails with the error

InjectTouchInput failure. GetLastError=87

in the console. I copied the zoom_in sequences from a couple of subtests of test_group_zoom.html, so it seems quite likely they are the same issue, so noting that here so that test can be enabled on Windows when this is fixed.

Blocks: 1636911

When I run some of the disabled tests locally I get the same "InjectTouchInput failure. GetLastError=87" error, so likely the same issue.

Error 87 is ERROR_INVALID_PARAMETER. So probably the arguments we're passing are not what the API expects.

FWIW, disabling


is enough to get test_group_zoom.html and test_group_zoom-2.html to pass locally and on try for me.

We start getting the error when we do the second touch point. The documenation for the api

says that it is passed an array of contacts:

"Array of POINTER_TOUCH_INFO structures that represents all contacts on the desktop. The screen coordinates of each contact must be within the bounds of the desktop."

The wording makes me think that maybe it expects every call to have the list of every active touch point, whereas we are calling it with single touch points and the first and second are intended to be active at the same time.

I also had to disable
for this same reason on Windows. (It's annotated with this bug number.)

Summary: Enable test_group_zoom.html on Windows → Enable test_group_zoom.html on Windows (InjectTouchInput failure)

I also had to disable
from bug 1707983 for this same reason on Windows. (It's annotated with this bug number.)

Severity: normal → S3
See Also: → 1875916
You need to log in before you can comment on or make changes to this bug.