web-platform/tests/battery-status/* all fail due to missing getBattery() function
Categories
(Core :: DOM: Device Interfaces, defect, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox78 | --- | fixed |
People
(Reporter: jmaher, Assigned: jmaher)
References
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
I was investigating a timeout in battery-status tests and realized that all the tests are expected fail and the few subtests that are not expected fail are error conditions or not related to battery-status.
It appears we have or had support for getBattery() as per bug 1050749.
Either this isn't supported by Firefox anymore, or the tests are broken.
I would like to determine if we support getBattery() on a canvas:
- supported (and we need to fix the tests)
- not [fully] supported, but on a roadmap in the next year (run the tests as 'backlog' on m-c only)
- no plans to support this (stop running the tests altogether)
Assignee | ||
Updated•5 years ago
|
Comment 1•5 years ago
|
||
(In reply to Joel Maher ( :jmaher ) (UTC-4) from comment #0)
I was investigating a timeout in battery-status tests and realized that all the tests are expected fail and the few subtests that are not expected fail are error conditions or not related to battery-status.
It appears we have or had support for getBattery() as per bug 1050749.
Either this isn't supported by Firefox anymore, or the tests are broken.
I would like to determine if we support getBattery() on a canvas:
- supported (and we need to fix the tests)
- not [fully] supported, but on a roadmap in the next year (run the tests as 'backlog' on m-c only)
- no plans to support this (stop running the tests altogether)
Hi Joel,
It looks like getBattery()
is supported in FF 43, so we can verify which testes should be enabled or disabled. We might have some unfinished features of Battery APIs. But, I don't think we would have other resource who can work on this recently. The original authors are inactive now.
Updated•5 years ago
|
Updated•5 years ago
|
Assignee | ||
Comment 2•5 years ago
|
||
:daoshengmu, thanks for the reply, we need to fix, disable, or reduce the timeout tests- as you indicated nobody has the expertise- is there someone that can figure it out this week?
Comment 3•5 years ago
|
||
I would guess we don't want to enable this for fingerprinting reasons, although I can't see a standards-position on it. My expectation is that we can mark it as backlog for now and try to figure out if we can move to not-implementing.
Comment 4•5 years ago
|
||
When running tests, please make sure dom.battery.enabled;true
[1]. If they still have uncertain cases, I would agree with :jgraham, we can put it as backlog
.
Assignee | ||
Comment 5•5 years ago
|
||
battery-status tests are not passing, move to backlog (tier-2)
Updated•5 years ago
|
Comment 7•5 years ago
|
||
bugherder |
Comment 8•5 years ago
|
||
Bug 1313580 removed getBattery() from the web in Firefox 52.
https://www.fxsitecompat.dev/en-CA/docs/2016/battery-status-api-has-been-removed/
Description
•