Closed
Bug 623800
Opened 14 years ago
Closed 14 years ago
default new contexts to a NULL compartment
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
blocking2.0 | --- | betaN+ |
People
(Reporter: gal, Assigned: gal)
Details
(Whiteboard: [softblocker], fixed-in-tracemonkey)
Attachments
(1 file)
1.00 KB,
patch
|
dmandelin
:
review+
|
Details | Diff | Splinter Review |
We currently default them to the default compartment, which seemed like a good idea at the time, but ends up being a total pain and it only hides bad bugs. With this change any bad selection of a compartment will cause a hard crash, so we can see it in opt builds.
Assignee | ||
Comment 1•14 years ago
|
||
Assignee: general → gal
Assignee | ||
Comment 2•14 years ago
|
||
jorendorff, should we take this for 4?
Assignee | ||
Comment 3•14 years ago
|
||
b? to get it on the radar, don't approve just yet please
blocking2.0: --- → ?
Comment 4•14 years ago
|
||
Andreas, have you decided what you want to do with this yet? (i.e., blocking status)
Assignee | ||
Comment 5•14 years ago
|
||
We should land it.
Assignee | ||
Updated•14 years ago
|
Attachment #501894 -
Flags: review?(dmandelin)
Assignee | ||
Updated•14 years ago
|
blocking2.0: ? → betaN+
Whiteboard: [softblocker]
Updated•14 years ago
|
Attachment #501894 -
Flags: review?(dmandelin) → review+
Assignee | ||
Comment 6•14 years ago
|
||
Whiteboard: [softblocker] → [softblocker], fixed-in-tracemonkey
Comment 7•14 years ago
|
||
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•