Closed Bug 700261 Opened 8 years ago Closed 8 years ago

Add .chargingTime/.dischargingTime support to hal code

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: mounir, Assigned: mounir)

References

Details

Attachments

(1 file)

Attached patch Patch v1Splinter Review
This doesn't do the backends implementations.
Attachment #572428 - Flags: review?(jones.chris.g)
Blocks: 700262
Blocks: 700263
Comment on attachment 572428 [details] [diff] [review]
Patch v1

dom::battery::kUnknownRemainingTime needs to be defined somewhere, right?
Attachment #572428 - Flags: review?(jones.chris.g) → review+
(In reply to Chris Jones [:cjones] [:warhammer] from comment #1)
> Comment on attachment 572428 [details] [diff] [review] [diff] [details] [review]
> Patch v1
> 
> dom::battery::kUnknownRemainingTime needs to be defined somewhere, right?

In the DOM patch.
Flags: in-testsuite?
Whiteboard: [needs review]
Comment on attachment 572428 [details] [diff] [review]
Patch v1

This has just landed in mozilla-inbound. Requesting aurora approval for API completeness in Firefox 10. It would be better to ship Firefox 10 with the entire API if we say it is present. The risk is very low given that there is no use of the API for the moment.
Attachment #572428 - Flags: approval-mozilla-aurora?
https://hg.mozilla.org/mozilla-central/rev/6918831fdd2f
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla11
Comment on attachment 572428 [details] [diff] [review]
Patch v1

It looks like this feature wasn't ready for 10. Please re-nominate if you have a good justification for trying to get this into 10 rather than just waiting for 11.
Attachment #572428 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora-
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.