Closed Bug 1754202 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 32747 - change in compute pressure API

Categories

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

task

Tracking

(firefox100 fixed)

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Arnaud Mandy <arnaud.mandy@intel.com> wrote:

change in compute pressure API

Moving the bucketing parameters from constructor to
observe() function.

This implies that parameter validity is checked when
observe() is called.

Adding disconnect() which replaces stop()
Adding unobserve(target), which is not yet implemented
on the browser side. It behaves just like disconnect() for now.

Change-Id: Id95870af9cf81bfc4c0d95cea2e048badf5aab26

Reviewed-on: https://chromium-review.googlesource.com/3447267
WPT-Export-Revision: 1f8cf47653545e68a008e23ee11f47e01ec078a7

Whiteboard: [wptsync downstream] → [wptsync downstream error]

CI Results

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

Total 13 tests and 1 subtests

Status Summary

Firefox

OK : 13
PASS : 1
FAIL : 36

Chrome

OK : 1
PASS : 32
FAIL : 4
ERROR : 12
NOTRUN: 1

Safari

OK : 13
PASS : 1
FAIL : 36

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/compute-pressure/compute_pressure_arguments.tentative.https.window.html
ComputePressureObserver constructor throws when cpuUtilizationThresholds is [-1]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor throws when cpuUtilizationThresholds is [0]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor throws when cpuUtilizationThresholds is [1]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor throws when cpuUtilizationThresholds is [2]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuUtilizationThresholds value [0.05]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuUtilizationThresholds value [0.1]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuUtilizationThresholds value [0.2]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuUtilizationThresholds value [0.5]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuUtilizationThresholds value [0.9]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuUtilizationThresholds value [0.95]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor throws when cpuSpeedThresholds is [-1]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor throws when cpuSpeedThresholds is [0]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor throws when cpuSpeedThresholds is [1]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor throws when cpuSpeedThresholds is [2]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuSpeedThresholds value [0.05]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuSpeedThresholds value [0.1]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuSpeedThresholds value [0.2]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuSpeedThresholds value [0.5]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuSpeedThresholds value [0.9]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor accepts cpuSpeedThresholds value [0.95]: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver constructor throws when cpuUtilizationThresholds has duplicates: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_basic.tentative.https.window.html
An active ComputePressureObserver calls its callback at least once: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_detached_iframe.tentative.https.html
ComputePressureObserver.observe() on detached frame rejects: FAIL (Chrome: PASS, Safari: FAIL)
ComputePressureObserver.stop() on detached frame returns: FAIL (Chrome: FAIL, Safari: FAIL)
Detaching frame while ComputePressureObserver.observe() settles: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_different_quantizations.tentative.https.window.html
ComputePressureObserver with a new quantization schema stops all other active observers: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_different_quantizations_across_iframes.tentative.https.window.html
ComputePressureObserver with a new quantization schema stops all other active observers: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_disconnect.tentative.https.window.html
Stopped ComputePressureObservers do not receive updates: FAIL (Chrome: FAIL, Safari: FAIL)
/compute-pressure/compute_pressure_disconnect_idempotent.tentative.https.window.html
Stopped ComputePressureObservers do not receive updates: FAIL (Chrome: FAIL, Safari: FAIL)
/compute-pressure/compute_pressure_disconnect_immediately.tentative.https.window.html
Stopped ComputePressureObservers do not receive updates: FAIL (Chrome: FAIL, Safari: FAIL)
/compute-pressure/compute_pressure_multiple.tentative.https.window.html
Three ComputePressureObserver instances with the same quantization schema receive updates: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_multiple_across_iframes.tentative.https.window.html
Three ComputePressureObserver instances in different iframes, but with the same quantization schema, receive updates: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_observe_idempotent.tentative.https.window.html
ComputePressureObserver.observe() is idempotent: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_observe_unobserve_failure.tentative.https.window.html
ComputePressureObserver.observe() requires a valid source: FAIL (Chrome: NOTRUN, Safari: FAIL)
ComputePressureObserver.unobserve() requires a valid source: FAIL (Chrome: PASS, Safari: FAIL)
/compute-pressure/compute_pressure_values.tentative.https.window.html
ComputePressureObserver quantizes utilization and speed separately: FAIL (Chrome: PASS, Safari: FAIL)

Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d458cd22598d
[wpt PR 32747] - compute pressure: Change in compute pressure API, a=testonly
https://hg.mozilla.org/integration/autoland/rev/86cdf9bffcdd
[wpt PR 32747] - Update wpt metadata, a=testonly
Whiteboard: [wptsync downstream error] → [wptsync downstream]
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
You need to log in before you can comment on or make changes to this bug.