Closed Bug 733652 Opened 8 years ago Closed 7 years ago

remove geolocation address

Categories

(Core :: DOM: Geolocation, defect)

x86
macOS
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla14

People

(Reporter: dougt, Assigned: dougt)

Details

(Keywords: addon-compat, dev-doc-complete)

Attachments

(1 file)

Attached patch patch v.1Splinter Review
Remove geolocation address from API.  Blog post forthcoming.
Attachment #603577 - Flags: review?(josh)
Comment on attachment 603577 [details] [diff] [review]
patch v.1

Wow, we should delete API features more often. Should we consider holding off on landing this until after the merge next week?
Attachment #603577 - Flags: review?(josh) → review+
Comment on attachment 603577 [details] [diff] [review]
patch v.1

it probably doesn't matter all that much.  The desktop doesn't support this now since GLS (the default location service for Firefox desktop) doesn't ever return an address.
https://hg.mozilla.org/mozilla-central/rev/691063f2a50e
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
Documentation updated:

https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsIDOMGeoPositionAddress

And mentioned on Firefox 14 for developers.
Just happened to visit this from the AMO blog.

This was backed out, so shouldn't be marked fixed.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
ed, thanks for double checking.  I have confirmed that this patch did stick and we have removed geolocation address.
Status: REOPENED → RESOLVED
Closed: 8 years ago7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.