1543337 - Unknown pointerType: [object String] "touch"

Status: RESOLVED FIXED

(Remote Protocol :: Marionette, defect, P3)

Description

[Alexei Barantsev]

Attachment: debug.log

User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/73.0.3683.86 Safari/537.36

Steps to reproduce:

Use trunk of Selenium Java binding and run this code:

WebDriver driver = new FirefoxDriver();
driver.get("http://localhost/"); // any site
WebElement element = driver.findElement(By.tagName("body")); // any element
new TouchActions(driver).singleTap(element).perform();

Actual results:

Exception thrown
InvalidArgumentException: Unknown pointerType: [object String] "touch"

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Thank you for reporting this Alexei! I noticed something similar lately when working with actions directly in Marionette but didn't notice that this was actually such a huge factor.

Here the excerpt from the log:

1554882024701 Marionette DEBUG 0 -> [0,4,"WebDriver:PerformActions",{"actions":[{"actions":[{"button":0,"type":"pointerDown"},{"button":0,"type":"pointerUp"}],"id":"touch screen","parameters":{"pointerType":"touch"},"type":"pointer"}]}]
1554882024703 Marionette DEBUG 0 <- [1,4,{"error":"invalid argument","message":"Unknown pointerType: [object String] "touch"","stacktrace":"WebDriverError@chro ... :545:3\nregisterSelf@chrome://marionette/content/listener.js:456:5\n@chrome://marionette/content/listener.js:1682:1\n"},null]

We have touch only for legacy actions right now, which we are going to remove via bug 1354578 so we should consider this as a blocker.

Comment 2

[Intermittent Failures Robot]

1 failures in 3031 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-central: 1

Platform breakdown:

• linux64-shippable: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1543337&startday=2019-04-22&endday=2019-04-28&tree=all

Comment 3

[Intermittent Failures Robot]

1 failures in 4730 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-central: 1

Platform breakdown:

• linux64: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1543337&startday=2019-05-20&endday=2019-05-26&tree=all

Comment 4

[Intermittent Failures Robot]

1 failures in 4337 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• mozilla-beta: 1

Platform breakdown:

• linux64-shippable-qr: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1543337&startday=2019-05-27&endday=2019-06-02&tree=all

Comment 5

[Intermittent Failures Robot]

2 failures in 5197 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 2

Platform breakdown:

• macosx1014-64-shippable: 1
• macosx1014-64: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1543337&startday=2019-06-03&endday=2019-06-09&tree=all

Comment 6

[Henrik Skupin [:whimboo][⌚️UTC+1]]

This bug shouldn't be used for the classification of intermittent failures. As such I filed bug 1562167 to move that off here.

Comment 7

[Intermittent Failures Robot]

1 failures in 4409 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• try: 1

Platform breakdown:

• android-em-7-0-x86_64: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1543337&startday=2019-06-24&endday=2019-06-30&tree=all

Comment 8

[Edwin Takahashi (:egao | infrequent contributor)]

:jgraham pointed me to this bug; this error is consistently observable when running web-platform-tests on the debian 10 image, as noted in https://bugzilla.mozilla.org/show_bug.cgi?id=1580401.

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&group_state=expanded&revision=13513111a9183259074f1cfeaf773f37146be166

The rate at which this error appears in debian 10 wpt is quite high; in some cases this message outnumbers the actual failures.

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+1]]

David, mind having a look if this bug is something we have to take care about? Looks like especially wpt on GeckoView is affected. See the latest added blocking bug.

Also I assume you actually don't work on this bug, so un-assigning you.

Comment 10

[David Burns :automatedtester]

I had been working on it slowly as it wasnt a priority

Comment 11

[David Burns :automatedtester]

Attachment: Bug 1543337: Add Touch pointer support to Marionette

Comment 12

[Henrik Skupin [:whimboo][⌚️UTC+1]]

(In reply to David Burns :automatedtester from comment #10)

I had been working on it slowly as it wasnt a priority

The attached patch is for pen but not for touch which this bug is actually about. Did you wanted to implement all at once, or is that just by accident the wrong patch? As it looks like it's getting more and more an annoyance factor especially for GeckoView based browsers. So we might have to address it at some point.

Comment 13

[David Burns :automatedtester]

I had it all but a rebase appears to have lost my tests and part of the patch. I will recreate it today

Comment 14

[David Burns :automatedtester]

Got it to pre-rebase state. It needs more tests which I will do next

Comment 15

[David Burns :automatedtester]

Attachment: Bug 1543337: Add Touch and Pen pointer support to Marionette

Bug 1543337: Improve error message when there is no click

Comment 16

[David Burns :automatedtester]

Masayuki,

I have added basic support for pen and touch to Marionette but the calls that it uses don't have the necessary event types being emited. Is there something like SendMouseEvent (https://searchfox.org/mozilla-central/source/dom/base/nsContentUtils.cpp#7741) that can work for pointer events?

Also, I appear to do something wrong with touch as those don't appear to be going through. Can you see any obvious mistakes in my patch. You can see the tests failing on https://treeherder.mozilla.org/#/jobs?repo=try&revision=8b61acfa599ae0af797586686b5833ce28c7d7d3

Comment 17

[Masayuki Nakano [:masayuki] (he/him)(JST, +0900)]

(In reply to David Burns :automatedtester from comment #16)

I have added basic support for pen and touch to Marionette but the calls that it uses don't have the necessary event types being emited. Is there something like SendMouseEvent (https://searchfox.org/mozilla-central/source/dom/base/nsContentUtils.cpp#7741) that can work for pointer events?

Pointer events are generated from mouse events and touch events. So, if you need pointer events, you need to use nsDOMWindowUtils::SendMouseEvent() or nsDOMWindowUtils::SendTouchEvent() (or their ToWindow(). If they don't work, I don't know the reason. There are some alternatives like nsDOMWindowUtils::SendNativeMouseMove(), nsDOMWindowUtils::SendNativeTouchPoint(), nsDOMWindowUtils::SendNativeTouchTap(), but they must have been designed for widget code testing. Therefore, the former should be useful.

Also, I appear to do something wrong with touch as those don't appear to be going through. Can you see any obvious mistakes in my patch. You can see the tests failing on https://treeherder.mozilla.org/#/jobs?repo=try&revision=8b61acfa599ae0af797586686b5833ce28c7d7d3

Hmm, I'm really not sure. Looks like the origin of coordinates do not match?

Comment 18

[David Burns :automatedtester]

Attachment: Bug 1543337: Add Touch and Pen pointer support to Marionette

Bug 1543337: Improve error message when there is no click

Comment 19

[Intermittent Failures Robot]

1 failures in 5015 pushes (0.0 failures/push) were associated with this bug in the last 7 days.

Repository breakdown:

• autoland: 1

Platform breakdown:

• linux1804-64-asan: 1

For more details, see:
https://treeherder.mozilla.org/intermittent-failures.html#/bugdetails?bug=1543337&startday=2020-03-30&endday=2020-04-05&tree=all

Comment 20

[Andrew McCreight [:mccr8]]

Can somebody work on fixing this, or at least changing something so we don't get the warning output flooding the TreeHerder suggestion box? Bug 1608101 is just filled with tons of mis-stars. The problem, as I'm sure you've seen, is that this error happens like a dozen times in a single test, so anybody trying to find the right failure to do a classification has to scroll through a lot of noise to find the actual error. If an intermittent failure bug like bug 1608101 happens to get the magic string in it, then it becomes the target for lots of incorrectly classfied oranges.

Here's an example: https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&selectedJob=298183024&revision=853b0e791775ce726149209092a003ed5f001b0c

Comment 21

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

I pushed a fix for the wpt log spam in Bug 1562167

Comment 22

[Henrik Skupin [:whimboo][⌚️UTC+1]]

To replace legacy actions with the webdriver equivalent having support for touch seems to be necessary. Maja, do you agree given by your recent investigations?

Comment 23

[Maja Frydrychowicz :impossibus (was :maja_zf) (needinfo me)]

Yes.

Comment 24

[Henrik Skupin [:whimboo][⌚️UTC+1]]

Note that with bug 1606061 fixed recently we got touch emulation support working for Fission. As such we can turn on touch emulation for devices that don't natively support it. And I'm fairly sure that this what we want here.

So basically we should give it another try to get it implemented. Also because it's the last blocker to get legacy actions removed.

Comment 25

[Chris Peterson [:cpeterson]]

Tracking marionette-fission-reserve bugs for Fission Future (post-MVP).

Comment 26

[Henrik Skupin [:whimboo][⌚️UTC+1]]

It should actually not block the remaining fission work, given that the APIs are implemented 1:1 in the JSWindowActor.

Comment 27

[Chris Peterson [:cpeterson]]

(In reply to Henrik Skupin (:whimboo) [⌚️UTC+2] from comment #26)

It should actually not block the remaining fission work, given that the APIs are implemented 1:1 in the JSWindowActor.

Cool. In that case, I will clear the Fission Milestone flag.

Comment 28

[Henrik Skupin [:whimboo][⌚️UTC+1]]

This requires an update for EventUtils.js which James is working on over on bug 1686666.

Comment 29

[James Graham [:jgraham]]

Attachment: Bug 1543337 - Refactor actions support,

Move from having lots of switch/case statements to dispatch the
correct actions to a more normal OO setup where each action type is
represented by a class with a static fromJSON method for construction,
and a dispatch() method for emitting the event.

The state is also passed around explictly rather than being stored in
a module global. This will allow us to have e.g. different state per
session.

Comment 30

[James Graham [:jgraham]]

Attachment: Bug 1543337 - Implement support for touch input in marionette,

Touch is a bit different to mouse inputs because there are often
multiple pointers (i.e. fingers) acting at the same time. It turns out
that Gecko wants a single call containing all the positions etc. of
the touch pointers for each event, rather than having a single call
per pointer. So we have to group the pointer actions from one tick
together.

Comment 31

[James Graham [:jgraham]]

Attachment: Bug 1543337 - Split touch pointer event into two,

For touch type pointers gecko doesn't support all the properties in pointer-events (and
it's not even very clear to me that they all make sense; how can one talk about the twist of
a finger, for example?). So split out a version of the tests that only test the properties
which are supported, so that we can regression-test the basics without failing due to missing
support at the engine layer for more complex properties.

Comment 32

[James Graham [:jgraham]]

Attachment: Bug 1543337 - Update pointer input support in webdriver crate,

Add all the additional pointer properties defined in spec to the
webdriver crate so they can be used by clients when supported by the
browser.

Comment 33

[James Graham [:jgraham]]

Attachment: Bug 1543337 - Update pointer input support in webdriver crate,

Add all the additional pointer properties defined in spec to the
webdriver crate so they can be used by clients when supported by the
browser.

Comment 34

[Phabricator Automation]

Comment on attachment 9264565 [details]
Bug 1543337 - Refactor actions support,

Revision D139119 was moved to bug 1773264. Setting attachment 9264565 [details] to obsolete.

Comment 35

[Phabricator Automation]

Comment on attachment 9274034 [details]
Bug 1543337 - Update pointer input support in webdriver crate,

Revision D144813 was moved to bug 1773265. Setting attachment 9274034 [details] to obsolete.

Comment 36

[James Graham [:jgraham]]

Attachment: Bug 1543337 - Change some `for (let _ of …)` to `for (const _ of …)`,

Comment 37

[James Graham [:jgraham]]

Attachment: Bug 1543337 - Allow multiple pointermove events in touch infrastructure tests,

At least in gecko, we are generating a pointermove event for each
pointer even when there was no actual change. This corresponds to the
behaviour of touch events, where the touchmove event includes all
active touch points.

Although this looks like it doesn't match the intent of the Pointer
Events spec, for an infrastructure test that's just checking if the
testdriver feature is supported, it seems better to allow the
discrepancy from the ideal behaviour rather than imply that the whole
feature is broken.

Comment 38

[James Graham [:jgraham]]

Attachment: Bug 1543337 - Ensure we pass integer coordinates to actions,

Comment 39

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a2a4ccd34675
Implement support for touch input in marionette, r=webdriver-reviewers,whimboo,jdescottes
https://hg.mozilla.org/integration/autoland/rev/dcf8d6675ecb
Ensure we pass integer coordinates to actions, r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/2982c3f1cee3
Change some `for (let _ of …)` to `for (const _ of …)`, r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/f0192382bd67
Split touch pointer event into two, r=webdriver-reviewers,whimboo,jdescottes
https://hg.mozilla.org/integration/autoland/rev/fcb9effe543d
Allow multiple pointermove events in touch infrastructure tests, r=webdriver-reviewers,whimboo

Comment 40

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/34804 for changes under testing/web-platform/tests

Comment 41

[bszekely]

Backed out for causing mutiple wpt failures.

Failure line 1 : TEST-UNEXPECTED-FAIL | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse | Inner frame mouse pointerover's button is -1 when mouse buttons are in released state. - assert_equals: Button attribute is -1 expected -1 but got 0
Failure log 1

Failure line 2 : TEST-UNEXPECTED-OK | /visual-viewport/scroll-event-order.html | expected ERROR
Failure log 2

Push with failures

Backout link

Failure 1

[task 2022-07-12T12:48:55.403Z] 12:48:55     INFO - TEST-START | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse
[task 2022-07-12T12:48:55.779Z] 12:48:55     INFO - {'actions': [{'type': 'none', 'actions': [{'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}], 'id': '0'}, {'type': 'pointer', 'actions': [{'type': 'pointerMove', 'x': 0, 'y': 0, 'origin': {'element-6066-11e4-a52e-4f735466cecf': '3c2aabc8-87da-4f74-b1f2-d65d8d5edc9d'}}, {'type': 'pointerDown', 'button': 0}, {'type': 'pointerUp', 'button': 0}], 'parameters': {'pointerType': 'mouse'}, 'id': 'mousePointer1'}]}
[task 2022-07-12T12:48:55.887Z] 12:48:55     INFO - {'actions': [{'type': 'none', 'actions': [{'type': 'pause', 'duration': 16}], 'id': '1'}, {'type': 'pointer', 'actions': [{'type': 'pointerMove', 'x': 0, 'y': 0, 'origin': 'viewport'}], 'parameters': {'pointerType': 'mouse'}, 'id': 'mousePointer1'}]}
[task 2022-07-12T12:48:55.963Z] 12:48:55     INFO - {'actions': [{'type': 'none', 'actions': [{'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}], 'id': '2'}, {'type': 'pointer', 'actions': [{'type': 'pointerMove', 'x': 0, 'y': 0, 'origin': {'element-6066-11e4-a52e-4f735466cecf': '5acbdc59-bc3c-4461-bf35-96b089d39dcd'}}, {'type': 'pointerDown', 'button': 0}, {'type': 'pointerUp', 'button': 0}], 'parameters': {'pointerType': 'mouse'}, 'id': 'mousePointer1'}]}
[task 2022-07-12T12:48:56.063Z] 12:48:56     INFO - {'actions': [{'type': 'none', 'actions': [{'type': 'pause', 'duration': 16}], 'id': '3'}, {'type': 'pointer', 'actions': [{'type': 'pointerMove', 'x': 0, 'y': 0, 'origin': 'viewport'}], 'parameters': {'pointerType': 'mouse'}, 'id': 'mousePointer1'}]}
[task 2022-07-12T12:48:56.390Z] 12:48:56     INFO - 
[task 2022-07-12T12:48:56.390Z] 12:48:56     INFO - TEST-FAIL | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave.fromElement value is null. - assert_equals: fromElement attribute value expected (object) null but got (undefined) undefined
[task 2022-07-12T12:48:56.390Z] 12:48:56     INFO - check_PointerEvent/</<@http://web-platform.test:8000/pointerevents/pointerevent_support.js:83:30
[task 2022-07-12T12:48:56.390Z] 12:48:56     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2590:25
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - check_PointerEvent/<@http://web-platform.test:8000/pointerevents/pointerevent_support.js:82:17
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - check_PointerEvent@http://web-platform.test:8000/pointerevents/pointerevent_support.js:57:7
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - checkPointerEventAttributes@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:80:35
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - run/</<@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:110:52
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - TEST-FAIL | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave.toElement attribute exists - assert_true: toElement attribute in pointerleave event expected true got false
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - check_PointerEvent/</<@http://web-platform.test:8000/pointerevents/pointerevent_support.js:65:24
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2590:25
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - check_PointerEvent/<@http://web-platform.test:8000/pointerevents/pointerevent_support.js:64:13
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - check_PointerEvent@http://web-platform.test:8000/pointerevents/pointerevent_support.js:57:7
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - checkPointerEventAttributes@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:80:35
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - run/</<@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:110:52
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - TEST-PASS | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave.toElement is readonly 
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - TEST-FAIL | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave.toElement IDL type object (JS type was undefined) - assert_true: toElement attribute of type object expected true got false
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - check_PointerEvent/</<@http://web-platform.test:8000/pointerevents/pointerevent_support.js:77:24
[task 2022-07-12T12:48:56.391Z] 12:48:56     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2590:25
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - check_PointerEvent/<@http://web-platform.test:8000/pointerevents/pointerevent_support.js:76:13
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - check_PointerEvent@http://web-platform.test:8000/pointerevents/pointerevent_support.js:57:7
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - checkPointerEventAttributes@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:80:35
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - run/</<@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:110:52
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - TEST-FAIL | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave.toElement value is null. - assert_equals: toElement attribute value expected (object) null but got (undefined) undefined
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - check_PointerEvent/</<@http://web-platform.test:8000/pointerevents/pointerevent_support.js:83:30
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2590:25
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - check_PointerEvent/<@http://web-platform.test:8000/pointerevents/pointerevent_support.js:82:17
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - check_PointerEvent@http://web-platform.test:8000/pointerevents/pointerevent_support.js:57:7
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - checkPointerEventAttributes@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:80:35
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - run/</<@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:110:52
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - TEST-PASS | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave.pressure value is valid 
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - TEST-PASS | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave properties for pointerType = mouse 
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - TEST-PASS | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave.isPrimary attribute is correct. 
[task 2022-07-12T12:48:56.392Z] 12:48:56     INFO - TEST-PASS | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse |  mouse pointerleave.pointerId should be the same as previous pointer events for this active pointer. 
[task 2022-07-12T12:48:56.393Z] 12:48:56     INFO - TEST-PASS | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse | Inner frame  mouse pointerover's type should be pointerover 
[task 2022-07-12T12:48:56.393Z] 12:48:56     INFO - TEST-UNEXPECTED-FAIL | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse | Inner frame  mouse pointerover's button is -1 when mouse buttons are in released state. - assert_equals: Button attribute is -1 expected -1 but got 0
[task 2022-07-12T12:48:56.393Z] 12:48:56     INFO - checkPointerEventAttributes/<@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:59:38
[task 2022-07-12T12:48:56.393Z] 12:48:56     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2590:25
[task 2022-07-12T12:48:56.393Z] 12:48:56     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2022-07-12T12:48:56.393Z] 12:48:56     INFO - checkPointerEventAttributes@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:58:25
[task 2022-07-12T12:48:56.393Z] 12:48:56     INFO - run/</<@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:119:52
[task 2022-07-12T12:48:56.395Z] 12:48:56     INFO - 
[task 2022-07-12T12:48:56.395Z] 12:48:56     INFO - TEST-UNEXPECTED-FAIL | /pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse | Inner frame  mouse pointerover's buttons is 0 when mouse buttons are in released state. - assert_equals: Buttons attribute is 0 expected 0 but got 1
[task 2022-07-12T12:48:56.395Z] 12:48:56     INFO - checkPointerEventAttributes/<@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:62:38
[task 2022-07-12T12:48:56.395Z] 12:48:56     INFO - Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:2590:25
[task 2022-07-12T12:48:56.395Z] 12:48:56     INFO - test@http://web-platform.test:8000/resources/testharness.js:628:30
[task 2022-07-12T12:48:56.395Z] 12:48:56     INFO - checkPointerEventAttributes@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:61:25
[task 2022-07-12T12:48:56.395Z] 12:48:56     INFO - run/</<@http://web-platform.test:8000/pointerevents/pointerevent_attributes_hoverable_pointers.html?mouse:119:52

Failure 2

[task 2022-07-12T13:00:36.299Z] 13:00:36     INFO - TEST-START | /visual-viewport/scroll-event-order.html
[task 2022-07-12T13:00:36.303Z] 13:00:36     INFO - Closing window 5817880e-391d-49d5-b5fd-c819acbe4e44
[task 2022-07-12T13:00:36.316Z] 13:00:36     INFO - Closing window 4772698f-4148-4c74-9daf-6c99231590d3
[task 2022-07-12T13:00:36.432Z] 13:00:36     INFO - {'actions': [{'type': 'none', 'actions': [{'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}, {'type': 'pause', 'duration': 16}], 'id': '0'}, {'type': 'pointer', 'actions': [{'type': 'pointerMove', 'x': 631, 'y': 419, 'origin': 'viewport'}, {'type': 'pointerDown', 'button': 0}, {'type': 'pointerMove', 'x': 631, 'y': 369, 'origin': 'viewport'}, {'type': 'pointerUp', 'button': 0}], 'parameters': {'pointerType': 'touch'}, 'id': 'finger1'}, {'type': 'pointer', 'actions': [{'type': 'pointerMove', 'x': 631, 'y': 519, 'origin': 'viewport'}, {'type': 'pointerDown', 'button': 0}, {'type': 'pointerMove', 'x': 631, 'y': 569, 'origin': 'viewport'}, {'type': 'pointerUp', 'button': 0}], 'parameters': {'pointerType': 'touch'}, 'id': 'finger2'}]}
[task 2022-07-12T13:00:36.563Z] 13:00:36     INFO - TEST-UNEXPECTED-OK | /visual-viewport/scroll-event-order.html | expected ERROR
[task 2022-07-12T13:00:36.563Z] 13:00:36     INFO - TEST-INFO expected ERROR | took 270ms
[task 2022-07-12T13:00:36.576Z] 13:00:36     INFO - PID 1144 | 1657630836575	Marionette	INFO	Stopped listening on port 50027
[task 2022-07-12T13:00:36.663Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1b97b900 - BlockShutdown: CanSend.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:3587
[task 2022-07-12T13:00:36.679Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1b97b900 - ShutDownProcess: Sent shutdown message.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:1791
[task 2022-07-12T13:00:36.679Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1ccf0600 - BlockShutdown: CanSend.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:3587
[task 2022-07-12T13:00:36.679Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1ccf0600 - ShutDownProcess: Sent shutdown message.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:1791
[task 2022-07-12T13:00:36.679Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1f9c2500 - BlockShutdown: CanSend.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:3587
[task 2022-07-12T13:00:36.679Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1f9c2500 - ShutDownProcess: Sent shutdown message.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:1791
[task 2022-07-12T13:00:36.688Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1b97b900 - ShutDownProcess: Closing channel.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:1826
[task 2022-07-12T13:00:36.688Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1b97b900 - RemoveShutdownBlockers: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:3689
[task 2022-07-12T13:00:36.690Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1ccf0600 - ShutDownProcess: Closing channel.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:1826
[task 2022-07-12T13:00:36.690Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1ccf0600 - RemoveShutdownBlockers: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:3689
[task 2022-07-12T13:00:36.691Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1f9c2500 - ShutDownProcess: Closing channel.: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:1826
[task 2022-07-12T13:00:36.692Z] 13:00:36     INFO - PID 1144 | [Parent 4496, Main Thread] WARNING: ContentParent: id=1f9c2500 - RemoveShutdownBlockers: file /builds/worker/checkouts/gecko/dom/ipc/ContentParent.cpp:3689
[task 2022-07-12T13:00:37.598Z] 13:00:37     INFO - Browser exited with return code 0
[task 2022-07-12T13:00:37.602Z] 13:00:37     INFO - Closing logging queue
[task 2022-07-12T13:00:37.603Z] 13:00:37     INFO - queue closed
[task 2022-07-12T13:00:37.735Z] 13:00:37     INFO - Application command: Z:\task_165762841631430\build\application\firefox\firefox.exe -marionette about:blank --wait-for-browser -profile C:\Users\task_165762841631430\AppData\Local\Temp\tmpwzrxtz3b
[task 2022-07-12T13:00:37.747Z] 13:00:37     INFO - PID 3180 | 1657630834101	Marionette	INFO	Marionette enabled
[task 2022-07-12T13:00:37.750Z] 13:00:37     INFO - PID 3180 | 1657630834104	Marionette	INFO	Listening on port 50094
[task 2022-07-12T13:00:37.751Z] 13:00:37     INFO - PID 3180 | console.warn: SearchSettings: "get: No settings file exists, new profile?" (new NotFoundError("Could not open the file at C:\\Users\\task_165762841631430\\AppData\\Local\\Temp\\tmp9d71cge6\\search.json.mozlz4", (void 0)))
[task 2022-07-12T13:00:37.752Z] 13:00:37     INFO - Starting runner
[task 2022-07-12T13:00:38.924Z] 13:00:38     INFO - TEST-START | /visual-viewport/viewport-no-resize-event-on-overflow-recalc.html

Comment 42

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b6f26244bf55
Implement support for touch input in marionette, r=webdriver-reviewers,whimboo,jdescottes
https://hg.mozilla.org/integration/autoland/rev/3fadd0a4cf4c
Ensure we pass integer coordinates to actions, r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/0c6c62853a16
Change some `for (let _ of …)` to `for (const _ of …)`, r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/9f7f9a7915fb
Split touch pointer event into two, r=webdriver-reviewers,whimboo,jdescottes
https://hg.mozilla.org/integration/autoland/rev/122a095250a4
Allow multiple pointermove events in touch infrastructure tests, r=webdriver-reviewers,whimboo

Comment 43

[Iulian Moraru]

Backed out for causing multiple wpt failures.

Push that got backed out
Push where the failures occurred

Failure log when it fails on interactionid-tap.html
Failure log when it fails on active-disabled.html
Failure log when it fails on mousemove_prevent_default_action.tentative.html
Failure log when it fails on caret-navigation-around-line-break.html
Failure log when it fails on modifying-selection-with-primary-mouse-button.tentative.html

Backout link

Comment 44

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7a7d21ba0298
Implement support for touch input in marionette, r=webdriver-reviewers,whimboo,jdescottes
https://hg.mozilla.org/integration/autoland/rev/55e76dcd4bde
Ensure we pass integer coordinates to actions, r=webdriver-reviewers,whimboo
https://hg.mozilla.org/integration/autoland/rev/3d69c7117ee9
Change some `for (let _ of …)` to `for (const _ of …)`, r=webdriver-reviewers,jdescottes
https://hg.mozilla.org/integration/autoland/rev/ac6f0d293d47
Split touch pointer event into two, r=webdriver-reviewers,whimboo,jdescottes
https://hg.mozilla.org/integration/autoland/rev/3e1db41f0e36
Allow multiple pointermove events in touch infrastructure tests, r=webdriver-reviewers,whimboo

Comment 45

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/7a7d21ba0298
https://hg.mozilla.org/mozilla-central/rev/55e76dcd4bde
https://hg.mozilla.org/mozilla-central/rev/3d69c7117ee9
https://hg.mozilla.org/mozilla-central/rev/ac6f0d293d47
https://hg.mozilla.org/mozilla-central/rev/3e1db41f0e36

Comment 46

[Web Platform Test Sync Bot (Matrix: #interop:mozilla.org)]

Upstream PR was closed without merging

Comment 47

[Henrik Skupin [:whimboo][⌚️UTC+1]]

(In reply to Web Platform Test Sync Bot (Matrix: #interop:mozilla.org) from comment #40)

Created web-platform-tests PR
https://github.com/web-platform-tests/wpt/pull/34804 for changes under
testing/web-platform/tests

Note that this PR got merged 3 days ago even with the bot not mentioning it here.

Comment 48

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/5348d13757d9
[wpt PR 34804] - [Gecko Bug 1543337] Implement support for touch input in marionette, a=testonly

Comment 49

[Iulian Moraru]

https://hg.mozilla.org/mozilla-central/rev/5348d13757d9