Closed Bug 1153988 Opened 5 years ago Closed 5 years ago

create nsNullPrincipals directly, rather than going through do_CreateInstance

Categories

(Core :: DOM: Core & HTML, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

There's a better way to create null principals than
do_CreateInstance("@mozilla.org/nullprincipal;1").  Let's do that and
save ourselves some XPCOM overhead.
Counting said that the CSSStyleSheet and nsNodeInfoManager ones were
(relatively) hot during tests; I replaced all instances to encourage people
when they cargo-cult things.
Attachment #8591823 - Flags: review?(bugs)
Attachment #8591823 - Flags: review?(bugs) → review+
Blah, that's what I get for gambling that the hadn't-started windows builds on try were going to be successful.
Flags: needinfo?(nfroyd)
https://hg.mozilla.org/mozilla-central/rev/2a81e53f2624
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.