Closed
Bug 727477
Opened 12 years ago
Closed 12 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•12 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•12 years ago
|
Assignee: nobody → doug.turner
Component: DOM: Device Interfaces → General
OS: Android → Linux
Hardware: All → x86_64
Comment 2•12 years ago
|
||
ROM Bug: http://code.google.com/p/android/issues/detail?id=22035
Reporter | ||
Comment 3•12 years ago
|
||
(sorry, I meant ICS, not IRC)
Comment 4•12 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•12 years ago
|
||
WONTFIX since our code works. Please subscribe to the google bug if you are interested.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → WONTFIX
Assignee | ||
Comment 6•12 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•12 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•12 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•12 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: 12 years ago → 12 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•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•