Closed Bug 578476 Opened 14 years ago Closed 14 years ago

Test for unexpected pass conditions in the JSON decoder

Categories

(Core :: DOM: Core & HTML, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla2.0b2

People

(Reporter: wesongathedeveloper, Assigned: wesongathedeveloper)

Details

Attachments

(1 file, 2 obsolete files)

This way, the tests always reflect the code's current behavior
Attached patch Patch (obsolete) — Splinter Review
should probably have done this in one bug (bug 578469) :/. oh well
Assignee: nobody → wesongathedeveloper
Status: NEW → ASSIGNED
Attachment #457145 - Flags: review?(jwalden+bmo)
Attached patch Patch (obsolete) — Splinter Review
remove tabs
Attachment #457145 - Attachment is obsolete: true
Attachment #457146 - Flags: review?(jwalden+bmo)
Attachment #457145 - Flags: review?(jwalden+bmo)
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+
Attached patch PatchSplinter Review
Attachment #457146 - Attachment is obsolete: true
Keywords: checkin-needed
(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
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b2
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: