DATA_CLONE_ERR has the wrong constant

RESOLVED FIXED in mozilla2.0b8

Status

()

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Ms2ger, Assigned: Ms2ger)

Tracking

Trunk
mozilla2.0b8
Points:
---

Firefox Tracking Flags

(blocking2.0 beta8+)

Details

(URL)

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

8 years ago
DATA_CLONE_ERR has code 25 assigned to it, but we use 18, which is SECURITY_ERR.
(Assignee)

Comment 1

8 years ago
Created attachment 491884 [details] [diff] [review]
Patch v1
Assignee: nobody → Ms2ger
Attachment #491884 - Flags: review?(jst)
(Assignee)

Updated

8 years ago
blocking2.0: --- → ?
Not blocking on this, and at this point I think I'd rather not change this any more for Firefox 4. This has been broken forever hasn't it? If so, the risk of taking this change now is higher than the gain IMO.
blocking2.0: ? → -
(Assignee)

Comment 3

8 years ago
This constant was added after b6, so, yes, you could say it's been wrong forever.
Comment on attachment 491884 [details] [diff] [review]
Patch v1

Oh, sorry, I thought this one had been shipped in 3.6 etc, but that's clearly not the case. In that case, I think we should fix this while we still can do so w/o running a risk of breaking a bunch of sites etc.
Attachment #491884 - Flags: review?(jst) → review+
Let's get this in for beta8 if at all possible then, to minimize exposing this incorrect value to the world...
blocking2.0: - → beta8+
(Assignee)

Comment 6

8 years ago
Created attachment 493346 [details] [diff] [review]
Patch for checkin

No problem, I gathered. Thanks for the review.
Attachment #491884 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/114966e5b168
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b8
You need to log in before you can comment on or make changes to this bug.