Closed Bug 353732 Opened 18 years ago Closed 18 years ago

Have a version of evalInSandbox that takes an nsIPrincipal, not a URI string

Categories

(Core :: XPConnect, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: bzbarsky, Assigned: mrbkap)

References

Details

Attachments

(1 file)

Once bug 351370 lands this should be pretty straightforward to do, and it could be useful on trunk where chrome script can get at principals.
Note that one can already pass in a window. Not sure why we require a window and not a general nsIScriptObjectPrincipal (say a document)...
Assignee: dbradley → nobody
Attached patch FixSplinter Review
I think this is all that's needed to fix this bug. Most of it was fixed by bug 351370.
Assignee: nobody → mrbkap
Status: NEW → ASSIGNED
Attachment #268437 - Flags: superreview?(jst)
Attachment #268437 - Flags: review?(jst)
Attachment #268437 - Flags: superreview?(jst)
Attachment #268437 - Flags: superreview+
Attachment #268437 - Flags: review?(jst)
Attachment #268437 - Flags: review+
Fix checked into trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Flags: in-testsuite?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: