The default bug view has changed. See this FAQ.

"warning: invalid manifest line element "pref(javascript.options.xml.content,true)" running jstests.py

RESOLVED FIXED in mozilla17

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jorendorff, Assigned: jorendorff)

Tracking

Other Branch
mozilla17
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

Comment hidden (empty)
(Assignee)

Comment 1

5 years ago
Created attachment 647326 [details] [diff] [review]
v1

E4X is disabled by default in the shell; you have to turn it on using
  options('allow_xml');

Up to now, the root shell.js script contained such an options() call, so E4X was effectively enabled for all JS tests, but only when they run in the shell.

With this patch, allow_xml is off by default in tests, and you can turn it on by using the magic comment

// |reftest| pref(javascript.options.xml.content,true)

The magic comment already worked in the jsreftest test harness, so it is already there for all E4X-using tests that run in the browser.

Up to now the shell test harness didn't know about this pref() thingy and it just spat out a warning. Now the shell test harness also honors this setting. I had to add the magic comment to a few tests that don't run in the browser.
Assignee: general → jorendorff
Attachment #647326 - Flags: review?(luke)

Updated

5 years ago
Attachment #647326 - Flags: review?(luke) → review+
(Assignee)

Comment 2

5 years ago
Created attachment 647572 [details] [diff] [review]
Rebased on top of terrence's patch in bug 778383

Carrying forward review.
Attachment #647326 - Attachment is obsolete: true
Attachment #647572 - Flags: review+
(Assignee)

Comment 3

5 years ago
Embarrassing:

https://hg.mozilla.org/integration/mozilla-inbound/rev/9fa54a1fc055

I think this is the first time Python's mutating += has bitten me. Pythons don't usually bite (they crush their prey).
https://hg.mozilla.org/mozilla-central/rev/25e961798919
https://hg.mozilla.org/mozilla-central/rev/9fa54a1fc055
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.