Re: CfC: Battery Last Call

On 22.11.2011, at 17.34, ext Mounir Lamouri wrote:

> My point was actually the opposite: the postulate we had since the beginning is that having no battery should be equivalent to having a fully charged battery so websites would be allowed to just check stuff as they want without caring about that detail. So we should have chargingTime = 0 when there is no battery. In addition, we wouldn't be able to know if the device has a battery or not.
> 
> To summarize, the requested changes are the following:
> * In 4.1, "charging", but that's just fixing an old mistake, see [1]:
> [ Represents if the system's battery is charging. The attribute must be set to true if the battery is charging or full, and set to false, if the battery is discharging, the implementation is unable to report the state, or there is no battery attached to the system, or otherwise. ]
> should be changed to:
> [ Represents if the system's battery is charging. The attribute must be set to false if the battery is discharging, and set to true, if the battery is charging, full, the implementation is unable to report the state, or there is no battery attached to the system, or otherwise. ]
> * In 4.1 "chargingTime":
> [ Represents the time remaining in seconds until the system's battery is fully charged. The attribute must be set to 0, if the battery is full, and to the value positive Infinity if the battery is discharging, the implementation is unable to report the remaining charging time, there is no battery attached to the system, or otherwise. ]
> should be changed to:
> [ Represents the time remaining in seconds until the system's battery is fully charged. The attribute must be set to 0, if the battery is full or there is no battery attached to the system, and to the value positive Infinity if the battery is discharging, the implementation is unable to report the remaining charging time, or otherwise. ]
> 
> [1] I realize that this is true according to the specs but I believe there is a mistake in the specs, we've never been talking of having charging = false when there is no battery.

The spec bugs are now fixed as per your proposal. Please check that we're now aligned with the implementation and ready to publish.

... and special thanks for the most excellent feedback :)

-Anssi

Received on Wednesday, 23 November 2011 08:15:17 UTC