remove 'countryCode' from nsIDOMGeoPositionAddress.idl

RESOLVED FIXED in mozilla11

Status

()

defect
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: myakura.web, Assigned: emorley)

Tracking

({addon-compat, dev-doc-complete})

Trunk
mozilla11
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Reporter

Description

8 years ago
User Agent: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/535.10 (KHTML, like Gecko) Chrome/17.0.945.0 Safari/535.10

Steps to reproduce:

Looked at http://mxr.mozilla.org/mozilla-central/source/dom/interfaces/geolocation/nsIDOMGeoPositionAddress.idl


Actual results:

found nsIDOMGeoPositionAddress.idl defines a property called 'countryCode' which isn't defined in the Geolocation v2 specification.


Expected results:

Per the spec, 'country' property "is specified using the two-letter [ISO 3166-1] code." so it seems 'countryCode' is pretty redundant. Rather than keeping undefined property or consult Geolocation WG to have it specified, I suggest removing it from Gecko.
Assignee

Comment 1

8 years ago
Posted patch Patch v1Splinter Review
Doug, can you confirm this is wanted please? :-)

https://tbpl.mozilla.org/?tree=Try&rev=403b9f4b60d2
Assignee: nobody → bmo
Attachment #581609 - Flags: feedback?(doug.turner)
Attachment #581609 - Flags: feedback?(doug.turner) → feedback+
Assignee

Updated

8 years ago
Attachment #581609 - Flags: review?(doug.turner)
Assignee

Updated

8 years ago
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #581609 - Flags: review?(doug.turner) → review+
Assignee

Comment 2

8 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/1d0ffc477fe8
Flags: in-testsuite-
Target Milestone: --- → mozilla11
Assignee

Comment 3

8 years ago
https://hg.mozilla.org/mozilla-central/rev/1d0ffc477fe8
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
OS: Windows 7 → All
Hardware: x86 → All
Resolution: --- → FIXED
I noted the attribute's removal at https://developer.mozilla.org/en/XPCOM_Interface_Reference/nsIDOMGeoPositionAddress , not sure if anything else needs to be done here.
I added a mention to Firefox 11 for developers and verified this property isn't used in any examples.

That article does need to move to the DOM reference with appropriate reformatting, but that's a separate issue.
You need to log in before you can comment on or make changes to this bug.