Closed
Bug 914792
Opened 11 years ago
Closed 11 years ago
Calls to JS shell builtin 'newGlobal' pass ignored 'new-compartment' option
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
mozilla26
People
(Reporter: jimb, Assigned: jimb)
Details
Attachments
(1 file)
294.42 KB,
patch
|
luke
:
review+
|
Details | Diff | Splinter Review |
Originally, the JS shell 'newGlobal' builtin function would return a global in a new compartment if passed the string 'new-compartment' as its argument. When SpiderMonkey changed to create a new compartment for every global, this became the only possible behavior for newGlobal, and it was changed to ignore string arguments. However, the string arguments to the calls weren't removed, and are still appearing via cargo-culting in new JS tests. The attached patch removes all these unused arguments from the calls to newGlobal in JS code in js/src.
Attachment #802511 -
Flags: review?(luke)
Updated•11 years ago
|
Attachment #802511 -
Flags: review?(luke) → review+
Assignee | ||
Comment 1•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/4299c9aa391b
Assignee: general → jimb
Status: NEW → ASSIGNED
Flags: in-testsuite-
Target Milestone: --- → mozilla26
Comment 2•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/4299c9aa391b
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•