Restrict BatteryManager to chrome script
Categories
(Core :: DOM: Device Interfaces, task, P3)
Tracking
()
People
(Reporter: jkt, Assigned: jkt)
References
Details
(Keywords: dev-doc-complete, site-compat)
Attachments
(1 file)
Comment 1•7 years ago
|
||
Comment 2•7 years ago
|
||
Assignee | ||
Comment 3•7 years ago
|
||
Updated•7 years ago
|
Assignee | ||
Comment 4•7 years ago
|
||
Comment 5•7 years ago
|
||
Comment 6•7 years ago
|
||
Comment 7•7 years ago
|
||
Comment 8•7 years ago
|
||
Comment 9•7 years ago
|
||
Comment 10•7 years ago
|
||
Comment 11•7 years ago
|
||
Updated•7 years ago
|
Updated•6 years ago
|
Comment 12•6 years ago
|
||
jkt, could you please land the [ChromeOnly] change and file a follow-up for further cleanup after extensions are sorted or some such? That at least ensures sites don't end up feature testing on this incorrectly or some such.
Assignee | ||
Comment 13•6 years ago
|
||
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Comment 14•6 years ago
|
||
Seeing if it just works tm, https://treeherder.mozilla.org/#/jobs?repo=try&revision=9e7f9c9413f200ab6f624c69959611996aabb7a6
Comment 15•6 years ago
|
||
Comment 16•6 years ago
|
||
Backed out changeset f065877ca676 (bug 1441976) for wpt failures at battery-status/battery-interface-idlharness.https.window.html
Backout: https://hg.mozilla.org/integration/autoland/rev/78ef6366dfaf5a8142a1d5894c35b243fb6ef133
Failure push: https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=f065877ca676168781ca11f205cef8dc8eedcf54
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=268654386&repo=autoland&lineNumber=1713
[task 2019-09-26T20:43:36.694Z] 20:43:36 INFO - TEST-START | /battery-status/battery-interface-idlharness.https.window.html
[task 2019-09-26T20:43:36.698Z] 20:43:36 INFO - Closing window 14
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO -
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - TEST-FAIL | /battery-status/battery-interface-idlharness.https.window.html | idl_test setup - promise_test: Unhandled rejection with value: object "TypeError: navigator.getBattery is not a function"
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - TEST-PASS | /battery-status/battery-interface-idlharness.https.window.html | Partial interface Navigator: original interface defined
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - TEST-UNEXPECTED-FAIL | /battery-status/battery-interface-idlharness.https.window.html | BatteryManager interface: existence and properties of interface object - assert_own_property: self does not have own property "BatteryManager" expected property "BatteryManager" missing
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - IdlInterface.prototype.assert_interface_object_exists@https://web-platform.test:8443/resources/idlharness.js:1332:24
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - IdlInterface.prototype.test_self/<@https://web-platform.test:8443/resources/idlharness.js:1543:14
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - Test.prototype.step@https://web-platform.test:8443/resources/testharness.js:1905:25
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - test@https://web-platform.test:8443/resources/testharness.js:544:30
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - self.subsetTestByKey@https://web-platform.test:8443/resources/idlharness.js:54:14
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - IdlInterface.prototype.test_self@https://web-platform.test:8443/resources/idlharness.js:1523:20
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - IdlInterface.prototype.test@https://web-platform.test:8443/resources/idlharness.js:1498:14
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - IdlArray.prototype.test@https://web-platform.test:8443/resources/idlharness.js:861:28
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO - idl_test/</<@https://web-platform.test:8443/resources/idlharness.js:3321:31
[task 2019-09-26T20:43:37.360Z] 20:43:37 INFO -
Comment 17•6 years ago
|
||
There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:jkt, could you have a look please?
For more information, please visit auto_nag documentation.
Updated•6 years ago
|
Comment 18•6 years ago
|
||
Comment 19•6 years ago
|
||
Updated the site compatibility note: https://www.fxsitecompat.dev/en-CA/docs/2016/battery-status-api-has-been-removed/
Comment 20•6 years ago
|
||
bugherder |
Assignee | ||
Updated•6 years ago
|
Comment 21•6 years ago
|
||
I've updated the compat data for BatteryManager
; I think this is all it needs:
Updated•6 years ago
|
Description
•