Closed Bug 1920254 Opened 1 month ago Closed 1 month ago

[wpt-sync] Sync PR 48298 - Convert compute_pressure_basic web tests to test_driver

Categories

(Testing :: web-platform-tests, task, P4)

Product:
Testing
Component:
web-platform-tests
Type:
task

Tracking

(firefox132 fixed)

Status:
RESOLVED FIXED
Milestone:
132 Branch
Tracking Flags:
Tracking Status
firefox132 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 48298 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/48298
Details from upstream follow.

Juha Vainio <juha.j.vainio@intel.com> wrote:

Convert compute_pressure_basic web tests to test_driver

The purpose of this change is start using WebDriver. WebDriver commands
for Compute Pressure are defined in
https://www.w3.org/TR/compute-pressure/#automation

Dedicated workers need to manipulate virtual pressure sources, but
testdriver calls can only be made in the embedded window, so the
existing solution of having '.any.js' files with META: global=window,dedicatedworker does not work anymore. The solution is
to use message passing so that workers make the embedder window invoke
the virtual pressure source calls and use WPT's variants concept to run
the same test under multiple configurations. Due to this limitation
'.any.js' files needs to be converted to '.window.js'. README.md has
been updated to include a detailed guide for creating tests.

WebDriver endpoints and virtual pressure sources work in Window and
Dedicated Worker scopes, but not shared worker ones: we store virtual
pressure source information in WebContentsUserData, which does not
integrate well with shared workers.

Bug: 347031400
Change-Id: I3cb482c0a689e84012735acd6bc9d33981a7d2f9
Reviewed-on: https://chromium-review.googlesource.com/5873130
WPT-Export-Revision: 78ceeebf3765f67235368b8929b5ab93e58c761f

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 5 subtests

Status Summary

Firefox

OK : 1
FAIL : 6
TIMEOUT: 1
ERROR : 1
NOTRUN : 3

Chrome

OK : 2
PASS : 10

Safari

OK : 1
FAIL : 6
TIMEOUT: 1
ERROR : 1
NOTRUN : 3

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

  • /compute-pressure/compute_pressure_basic.https.window.html?globalScope=dedicated_worker [wpt.fyi]: ERROR (Chrome: OK, Safari: ERROR)
    • Return NotSupportedError when calling observer(): FAIL (Chrome: PASS, Safari: FAIL)
    • Basic functionality test: TIMEOUT (Chrome: PASS, Safari: TIMEOUT)
    • Removing observer before observe() resolves works: NOTRUN (Chrome: PASS, Safari: NOTRUN)
    • Calling observe() multiple times works: NOTRUN (Chrome: PASS, Safari: NOTRUN)
    • Starting a new observer after an observer has started works: NOTRUN (Chrome: PASS, Safari: NOTRUN)
  • /compute-pressure/compute_pressure_basic.https.window.html?globalScope=window [wpt.fyi]
    • Return NotSupportedError when calling observer(): FAIL (Chrome: PASS, Safari: FAIL)
    • Basic functionality test: FAIL (Chrome: PASS, Safari: FAIL)
    • Removing observer before observe() resolves works: FAIL (Chrome: PASS, Safari: FAIL)
    • Calling observe() multiple times works: FAIL (Chrome: PASS, Safari: FAIL)
    • Starting a new observer after an observer has started works: FAIL (Chrome: PASS, Safari: FAIL)
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d13961669a93 [wpt PR 48298] - Convert compute_pressure_basic web tests to test_driver, a=testonly https://hg.mozilla.org/integration/autoland/rev/d45864590e0b [wpt PR 48298] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/d13961669a93
https://hg.mozilla.org/mozilla-central/rev/d45864590e0b

Status: NEW → RESOLVED
Closed: 1 month ago
status-firefox132: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
You need to log in before you can comment on or make changes to this bug.