Closed
Bug 727477
Opened 13 years ago
Closed 13 years ago
Battery API: On Galaxy Nexus, the Battery API reports "charging" and "full" even if not full and not plugged
Categories
(Core Graveyard :: Widget: Android, defect)
Tracking
(firefox10 wontfix, firefox11 affected, firefox12 affected, firefox13 fixed)
RESOLVED
FIXED
mozilla13
People
(Reporter: paul, Assigned: mounir)
Details
(Whiteboard: mwc-demo)
Attachments
(1 file)
2.70 KB,
patch
|
dougt
:
review+
|
Details | Diff | Splinter Review |
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).
Assignee | ||
Comment 1•13 years ago
|
||
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?
Component: General → DOM: Device Interfaces
OS: Linux → Android
QA Contact: general → device-interfaces
Hardware: x86_64 → All
Updated•13 years ago
|
Assignee: nobody → doug.turner
Component: DOM: Device Interfaces → General
OS: Android → Linux
Hardware: All → x86_64
Comment 2•13 years ago
|
||
Reporter | ||
Comment 3•13 years ago
|
||
(sorry, I meant ICS, not IRC)
Comment 4•13 years ago
|
||
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•13 years ago
|
||
WONTFIX since our code works. Please subscribe to the google bug if you are interested.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → WONTFIX
Assignee | ||
Comment 6•13 years ago
|
||
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.
Assignee: doug.turner → mounir
Status: RESOLVED → REOPENED
OS: Linux → Android
Hardware: x86_64 → All
Resolution: WONTFIX → ---
WebAPI Web app includes the battery API in the demo. Adding the whiteboard mwc-demo because of this.
Whiteboard: mwc-demo
Assignee | ||
Comment 8•13 years ago
|
||
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.
Attachment #599181 -
Flags: review?(doug.turner)
Comment 9•13 years ago
|
||
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.
Attachment #599181 -
Flags: review?(doug.turner) → review+
Assignee | ||
Comment 10•13 years ago
|
||
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?
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
status-firefox10:
--- → wontfix
status-firefox11:
--- → affected
status-firefox12:
--- → affected
status-firefox13:
--- → fixed
Component: General → Widget: Android
QA Contact: device-interfaces → android
Resolution: --- → FIXED
Target Milestone: --- → mozilla13
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•