Re: Feedback about Battery Status Events

On 10/18/2011 01:15 AM, Jonas Sicking wrote:
> I think the idea was to return null on devices which don't use battery
> at all. On my desktop machine there is no way I'm going to hot plug a
> battery and start carrying it away (I probably wouldn't get past the
> downstairs security guards anyway, despite 5 years of ice hockey).

And how can you know that? Do we have a sure way to know if a system is 
a laptop or a desktop? Off hand, I would say a desktop doesn't have a 
battery and a laptop has one... I don't see how to differentiate a 
desktop with a battery-less laptop. In addition, it might increase the 
risk of fingerprinting with a desktop returning null and a laptop 
returning a BatteryManager with level=1.0 and state="charging".

> On a device which has battery, but where one isn't currently plugged
> in it seems appropriate to return 0 for level of charge and true for
> charging. If a battery which is charged to 50% is plugged in simply
> change the charge level to 50% and fire the appropriate events.

It would be probably be better to have level=1.0 and state="charging" 
when the battery is removed. That way, script will not disable stuff 
thinking the device power is low.

--
Mounir

Received on Tuesday, 18 October 2011 08:58:10 UTC