Test for unexpected pass conditions in the JSON decoder

RESOLVED FIXED in mozilla2.0b2

Status

()

Core
DOM
--
enhancement
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Saint Wesonga, Assigned: Saint Wesonga)

Tracking

Trunk
mozilla2.0b2
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

8 years ago
This way, the tests always reflect the code's current behavior
(Assignee)

Comment 1

8 years ago
Created attachment 457145 [details] [diff] [review]
Patch

should probably have done this in one bug (bug 578469) :/. oh well
Assignee: nobody → wesongathedeveloper
Status: NEW → ASSIGNED
Attachment #457145 - Flags: review?(jwalden+bmo)
(Assignee)

Comment 2

8 years ago
Created attachment 457146 [details] [diff] [review]
Patch

remove tabs
Attachment #457145 - Attachment is obsolete: true
Attachment #457146 - Flags: review?(jwalden+bmo)
Attachment #457145 - Flags: review?(jwalden+bmo)

Comment 3

8 years ago
Comment on attachment 457146 [details] [diff] [review]
Patch

Make that do_throw instead.  Thanks for making more work for me when I land bug 564621, unless I somehow improbably manage to beat this to landing.  :-P
Attachment #457146 - Flags: review?(jwalden+bmo) → review+
(Assignee)

Comment 4

8 years ago
Created attachment 457214 [details] [diff] [review]
Patch
Attachment #457146 - Attachment is obsolete: true
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
(Assignee)

Comment 5

8 years ago
(In reply to comment #3)
> ... Thanks for making more work for me when I land bug
> 564621, ...
anytime :D
http://hg.mozilla.org/mozilla-central/rev/83550fdf1b23
Status: ASSIGNED → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b2
You need to log in before you can comment on or make changes to this bug.