Closed
Bug 1036517
Opened 11 years ago
Closed 11 years ago
[geolocation-stumbler] Remove old stumbling code from GeckoApp.java
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 33
People
(Reporter: garvan, Assigned: nalexander)
References
Details
Attachments
(1 file)
|
14.46 KB,
patch
|
blassey
:
review+
|
Details | Diff | Splinter Review |
This code is no longer relevant. Should be trivial to remove.
|
Assignee | |
Comment 1•11 years ago
|
||
I kept the existing preference UI since the new stumbler implementation
intends to use it. I kept the pref listener since Bug 1036508 may elect
to keep using the existing Gecko pref.
blassey: you landed the initial code, so perhaps you're the right
reviewer. Nothing touches the DOM geo API, but just in case I
verified that my location was reported to Google Maps correctly after
a build.
Attachment #8455834 -
Flags: review?(blassey.bugs)
I should apologize for disparaging this code, sorry. Looking at it again now, seems reasonable, just looks like requirements moved forward and the code never got updated.
|
||
Updated•11 years ago
|
Attachment #8455834 -
Flags: review?(blassey.bugs) → review+
|
|
Assignee | |
Comment 3•11 years ago
|
||
Assignee: nobody → nalexander
Status: NEW → ASSIGNED
|
||
Comment 4•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 33
|
||
Updated•4 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
•