Closed Bug 760709 Opened 12 years ago Closed 12 years ago

Isolate E4X jittests

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla16

People

(Reporter: terrence, Assigned: terrence)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch v0 (obsolete) — Splinter Review
With --disable-e4x, several jit-tests fail.  This reworks these tests to not use E4X, or if they are E4X specific, moves them to a separate e4x directory so they are easy to visually filter.

Most of these were fairly straightforward, however, I'm not 100% certain about the following tests and would like a second opinion:
  bug651966.js
  bug737251.js
  bug738846.js
Attachment #629369 - Flags: review?(jorendorff)
Attachment #629369 - Flags: review?(jorendorff)
This is incredibly tedious, so I'm going to split it into two parts.  This first part is just the tests that are specific to E4X -- e.g. the messed up moves from the last patch.
Attachment #629369 - Attachment is obsolete: true
Attachment #629969 - Flags: review?(jorendorff)
Attachment #629969 - Flags: review?(jorendorff) → review+
https://hg.mozilla.org/mozilla-central/rev/ad8edddea8d3

(Merged by Ed Morley)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla16
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: