nsISound::Play shouldn't pass system principal to loadinfo

NEW
Unassigned

Status

()

Core
Widget: Cocoa
P4
normal
4 years ago
2 years ago

People

(Reporter: tanvi, Unassigned)

Tracking

(Blocks: 2 bugs)

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: tpi:+)

(Reporter)

Description

4 years ago
https://mxr.mozilla.org/mozilla-central/source/widget/cocoa/nsSound.mm#59 calls NS_NewStreamLoader().  When we add the loadInfo to this channel in bug 1038756, we set the loadingPrincipal to systemPrincipal.

Per https://bugzilla.mozilla.org/show_bug.cgi?id=1038756#c171, Gecko code doesn't call this API.  If addons aren't using this API, we can remove it.  If they are, then this bug is to figure out how to get them to pass the right loadingPrincipal and/or fallback to something more restrictive than systemPrincipal (maybe we can pass nullPrincipal).
(Reporter)

Updated

3 years ago
Blocks: 1143922

Updated

2 years ago
Priority: -- → P4
Whiteboard: tpi:+
You need to log in before you can comment on or make changes to this bug.