Closed
Bug 410851
Opened 17 years ago
Closed 17 years ago
Expose faster version of GetSubjectPrincipal().
Categories
(Core :: Security: CAPS, defect)
Core
Security: CAPS
Tracking
()
RESOLVED
FIXED
mozilla1.9beta3
People
(Reporter: jst, Assigned: jst)
References
Details
(Keywords: perf)
Attachments
(1 file)
(deleted),
patch
|
mrbkap
:
review+
mrbkap
:
superreview+
|
Details | Diff | Splinter Review |
Right now, callers of GetSubjectPrincipal() that have the right JSContext at hand already don't have a way to pass that in to caps when getting the subject principal. In stead, we end up spending time looking up the current context etc when we really don't need to. This is especially important in IsWrapperSameOrigin() in the cross origin wrapper code. See attached patch for a fix.
Flags: blocking1.9+
Attachment #295435 -
Flags: superreview?(mrbkap)
Attachment #295435 -
Flags: review?(mrbkap)
Updated•17 years ago
|
Version: unspecified → Trunk
Updated•17 years ago
|
Attachment #295435 -
Flags: superreview?(mrbkap)
Attachment #295435 -
Flags: superreview+
Attachment #295435 -
Flags: review?(mrbkap)
Attachment #295435 -
Flags: review+
Assignee | ||
Comment 1•17 years ago
|
||
Fix checked in.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•