Closed
Bug 712576
Opened 14 years ago
Closed 14 years ago
Add GPS-A support on android
Categories
(Firefox for Android Graveyard :: General, enhancement, P3)
Tracking
(firefox11 fixed, firefox12 fixed, fennec11+)
RESOLVED
FIXED
Firefox 12
People
(Reporter: pispo1, Assigned: dougt)
References
Details
Attachments
(1 file)
1.59 KB,
patch
|
jdm
:
review+
akeybl
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/535.11 (KHTML, like Gecko) Chrome/17.0.963.12 Safari/535.11
Steps to reproduce:
Terminal: Google Samsung Galaxy Nexus with Android Sandwich
In Firefox Mobile, go to http://html5demos.com/geo.
In Native Browser, go to http://html5demos.com/geo.
Actual results:
In Firefox Mobile:
If the GPS-D (normal GPS) has acquired the minimun number of satellites (4), it works. In other case it fails.
In Native Browser: It works without the acquistion of number of satellites (without GPS-D). It is supposed that GPS-A is used.
Expected results:
The same behaviour of Native Browser is expected.
I cannot find the exact line in the code but it is related with the usage of LocationManager.GPS_PROVIDER (LocationManager.NETWORK_PROVIDER should be added).
Reporter | ||
Updated•14 years ago
|
OS: Windows 7 → Android
Hardware: x86_64 → ARM
Reporter | ||
Updated•14 years ago
|
Severity: normal → enhancement
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → doug.turner
Priority: -- → P3
Assignee | ||
Updated•14 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Updated•14 years ago
|
tracking-fennec: --- → 11+
Assignee | ||
Comment 2•14 years ago
|
||
i am not working on these right now. resetting assignee.
Assignee: doug.turner → nobody
Assignee | ||
Updated•14 years ago
|
Assignee: nobody → doug.turner
Assignee | ||
Comment 4•14 years ago
|
||
Attachment #588535 -
Flags: review?(josh)
Comment 5•14 years ago
|
||
Comment on attachment 588535 [details] [diff] [review]
patch v.1
Seems reasonable.
Attachment #588535 -
Flags: review?(josh) → review+
Comment 6•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 12
Assignee | ||
Updated•14 years ago
|
Attachment #588535 -
Flags: approval-mozilla-aurora?
Comment 7•14 years ago
|
||
Comment on attachment 588535 [details] [diff] [review]
patch v.1
[Triage Comment]
Mobile only - approved for Aurora.
Attachment #588535 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Updated•14 years ago
|
status-firefox11:
--- → fixed
status-firefox12:
--- → fixed
Updated•5 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•