Remove nsISecurityCheckedComponent from videocontrols

RESOLVED FIXED in mozilla8

Status

()

RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Dolske, Assigned: Dolske)

Tracking

unspecified
mozilla8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
Created attachment 543818 [details] [diff] [review]
Patch v.1

There's been some prior discussion of this interface in the past -- bug 470852, and notably bug 475864. As I understand things (poorly ;), this code isn't actually doing anything now, and I recall bz saying something long ago to the effect of we should be able to remove it once we're ok with the then-new quickstubs code.
Attachment #543818 - Flags: review?(bzbarsky)
Comment on attachment 543818 [details] [diff] [review]
Patch v.1

I don't remember the security implications here very well.  :(  Blake, can you take a look?
Attachment #543818 - Flags: review?(bzbarsky) → review?(mrbkap)
Comment on attachment 543818 [details] [diff] [review]
Patch v.1

I have vague memories of the conversations around this. I'm pretty sure that because we removed the native anonymous content check in nsScriptSecurityManager, this implementation of nsISecurityCheckedComponent is no longer needed.
Attachment #543818 - Flags: review?(mrbkap) → review+
http://hg.mozilla.org/mozilla-central/rev/5c7fc05dc9e3
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.