decomtaminate nsIScriptObjectPrincipal

RESOLVED FIXED

Status

()

Core
DOM
RESOLVED FIXED
14 years ago
14 years ago

People

(Reporter: Brian Ryner (not reading), Assigned: Brian Ryner (not reading))

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

14 years ago
This will let us remove the duplicate GetPrincipal() methods on the Document
implementations.
(Assignee)

Comment 1

14 years ago
Created attachment 146310 [details] [diff] [review]
patch

Probably the most interesting part here is the fix to nsCrypto.cpp.  When I was
looking at this code, I found that the braces didn't actually match the
indentation or the way the code is intended to work.
(Assignee)

Comment 2

14 years ago
Comment on attachment 146310 [details] [diff] [review]
patch

Also, I think the last change in nsCrypto.cpp plugs a leak.
Attachment #146310 - Flags: superreview?(jst)
Attachment #146310 - Flags: review?(jst)
Comment on attachment 146310 [details] [diff] [review]
patch

r+sr=jst
Attachment #146310 - Flags: superreview?(jst)
Attachment #146310 - Flags: superreview+
Attachment #146310 - Flags: review?(jst)
Attachment #146310 - Flags: review+
(Assignee)

Comment 4

14 years ago
checked in.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.