727477 - Battery API: On Galaxy Nexus, the Battery API reports "charging" and "full" even if not full and not plugged

Status: RESOLVED FIXED

(Core Graveyard :: Widget: Android, defect)

Description

[Paul Rouget [:paul]]

I use this code: http://jsbin.com/ahovij/3

It works on HTC Desire HD + IRC.
On Galaxy Nexus, I get "level: 1, charging: true;" what ever the status of the battery is (plugged on not plugged, charged or not charged).

Comment 1

[Mounir Lamouri (:mounir)]

I've been told that by someone from dev engagement but I don't have a Galaxy Nexus.

What do you mean by HTC Desire HD + IRC?

Comment 2

[Doug Turner (:dougt)]

ROM Bug:
   http://code.google.com/p/android/issues/detail?id=22035

Comment 3

[Paul Rouget [:paul]]

(sorry, I meant ICS, not IRC)

Comment 4

[Robert Nyman]

Yes, I've seen the same thing on Galaxy Nexus, and like Doug pointed out, it seems to be due to a bug at their end.

Comment 5

[Doug Turner (:dougt)]

WONTFIX since our code works.  Please subscribe to the google bug if you are interested.

Comment 6

[Mounir Lamouri (:mounir)]

I prefer to reopen it because I guess there is a way to get the battery information on this phone a way or another given that likely the battery widget is working.
I will buy a Galaxy Nexus next week when back in Paris and have a look at that. Reopening and assigning to myself for that.

Comment 7

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

WebAPI Web app includes the battery API in the demo.  Adding the whiteboard mwc-demo because of this.

Comment 8

[Mounir Lamouri (:mounir)]

Attachment: Patch

This should work around the low level issue.

BTW, it's not an ICS issue. I tried with a Nexus S with ICS and things were working nicely.

Comment 9

[Doug Turner (:dougt)]

Comment on attachment 599181 [details] [diff] [review]
Patch

Review of attachment 599181 [details] [diff] [review]:
-----------------------------------------------------------------

if there are more devices that need to be special cased, we can factor that test out into its own function.

Comment 10

[Mounir Lamouri (:mounir)]

Pushed to m-c:
https://hg.mozilla.org/mozilla-central/rev/f754a314d61d

Should we take this fix to Firefox 11 and/or 12 given that it's really safe and will prevent people from complaining that this new feature we might advertise doesn't work on their phones?