526317 - Remove nsIDOMGeoPositionError::message

Status: RESOLVED FIXED

(Core :: DOM: Geolocation, defect)

Description

[Doug Turner (:dougt)]

At the geolocation face to face, we discussed and agreed that the message attribute of the position error should be removed.

Comment 1

[Doug Turner (:dougt)]

Attachment: patch v.1

This also covers bug 526317.

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

(In reply to comment #1)
> This also covers bug 526317.
Wrong bug#?

Comment 3

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 412132 [details] [diff] [review]
patch v.1

 
>-[scriptable, uuid(1B493214-4E58-4A40-AA4C-1AB70C6DDBEC)]
>+[scriptable, uuid(AD9FA4C8-EC71-4B2D-8294-9ADF06DDEC32)]
> interface nsIDOMGeoPositionError : nsISupports
> {
>-  const unsigned short UNKNOWN_ERROR = 0;
Why are you removing this?
It is still mentioned in http://dev.w3.org/geo/api/spec-source.html#position_error_interface


If WG has decided to remote UNKNOWN_ERROR, then r=me.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

I guess you meant Bug 526320

Comment 5

[Doug Turner (:dougt)]

http://hg.mozilla.org/mozilla-central/rev/c3924062e8bc

Comment 6

[Doug Turner (:dougt)]

smaug: andrei has not updated the draft spec yet.  I will ping him.

Comment 7

[Doug Turner (:dougt)]

http://hg.mozilla.org/releases/mozilla-1.9.2/rev/26dcde501678

Comment 8

[Eric Shepherd [:sheppy]]

Doc updated:

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

Comment 9

[Chris Peterson [:cpeterson]]

dougt, the 2012 W3C spec's PositionError still includes a `message` attribute:

"The `message` attribute must return an error message describing the details of the error encountered."

http://www.w3.org/TR/2012/PR-geolocation-API-20120510/#position_error_interface