1642163 - 3 web-locks/ tests are expected TIMEOUT

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

I find that these tests in web-locks/ are expected timeout:
/web-locks/frames.tentative.https.html
/web-locks/query-ordering.tentative.https.html
/web-locks/workers.tentative.https.html

I have verified these timeout locally on my windows 10 instance.

here is the error I see in the console:
/web-locks/frames.tentative.https.html:
JavaScript error: https://web-platform.test:8443/web-locks/resources/iframe.html, line 18: TypeError: can't access property "request", navigator.locks is undefined

/web-locks/query-ordering.tentative.https.html:
JavaScript error: https://web-platform.test:8443/web-locks/resources/iframe.html, line 43: TypeError: can't access property "request", navigator.locks is undefined

/web-locks/workers.tentative.https.html:
TypeError: can't access property "request", navigator.locks is undefined
https://web-platform.test:8443/web-locks/resources/worker.js:15:0

these are also missing the proper bugzilla component.

:jgraham, can you help find a triage owner for these tests so we can start looking into if we support web-locks/*tentative.https.html tests or how we can fix the tests.

Comment 1

[James Graham [:jgraham]]

Attachment: Bug 1642163 - Make Web Locks tests check the API is implemented,

This avoids needless timeouts

Comment 2

[James Graham [:jgraham]]

Added a patch that will stop the timeouts. https://github.com/mozilla/standards-positions/issues/64 seems to be the standards-position for this; :asuth might know if we're planning to work on it in the foreseeable future.

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

It's on the short-list for implementation, but 2020 H2 OKR scheduling is still to happen in the future.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b9875bea79ca
Make Web Locks tests check the API is implemented, r=annevk

Comment 5

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

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

Comment 6

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

Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.

Comment 7

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

Upstream PR merged by moz-wptsync-bot

Comment 8

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/b9875bea79ca