Closed Bug 1473231 Opened 7 years ago Closed 7 years ago

Update test262 (Jul 04 2018 edition)

Categories

(Core :: JavaScript Engine, enhancement)

enhancement
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla63
Tracking Status
firefox63 --- fixed

People

(Reporter: anba, Assigned: anba)

Details

Attachments

(3 files)

No description provided.
Update the importer to handle new "flag" and "feature" options and remove the temporary workaround for the Atomics tests (the upstream issue has been fixed). Experimental Intl APIs are not enabled by default, even not in Nightly, instead the |addIntlExtras| shell function needs to be called to add the experimental APIs to the Intl object. Add a helper file and include it using the importer for Intl.RelativeTimeFormat tests. (The same approach was used when Intl.PluralRules wasn't enabled by default.)
Attachment #8989811 - Flags: review?(sphink)
The actual test262 update.
Attachment #8989813 - Flags: review?(sphink)
Updates the exclusion file after the test262 update. I've re-enabled the currently disabled Atomics tests because <https://github.com/tc39/test262/pull/1617> should have fixed the observed intermittent failures.
Attachment #8989815 - Flags: review?(sphink)
Comment on attachment 8989811 [details] [diff] [review] bug1473231-part1-update-importer.patch Review of attachment 8989811 [details] [diff] [review]: ----------------------------------------------------------------- You're awesome. Thank you.
Attachment #8989811 - Flags: review?(sphink) → review+
Attachment #8989813 - Flags: review?(sphink) → review+
Attachment #8989815 - Flags: review?(sphink) → review+
Pushed by cbrindusan@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/3f898765faf5 Part 1: Update test262 importer to handle new feature flags. r=sfink https://hg.mozilla.org/integration/mozilla-inbound/rev/af8711af6fef Part 2: Update test262 - Jul 04 2018 edition. r=sfink https://hg.mozilla.org/integration/mozilla-inbound/rev/cf8b3521ee57 Part 3: Update test262 exclusions. r=sfink
Keywords: checkin-needed
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: