Closed Bug 826410 Opened 11 years ago Closed 11 years ago
HTMLMedia Element is missing ns IAudio Channel Agent Callback in its QI impl
This leads to assertions in b2g debug builds.
Comment on attachment 697593 [details] [diff] [review] Fix We should also add a chrome caller check to nsHTMLMediaElement::CanPlayChanged() to make sure it can never be called by untrusted script. Please add that here as well so web pages can't interfere here and re-request approval on that patch.
Comment on attachment 697593 [details] [diff] [review] Fix https://hg.mozilla.org/integration/mozilla-inbound/rev/d1905a0ba1ed (with jst's requested change).
Attachment #697593 - Flags: checkin+
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla20
Bug caused by (feature/regressing bug #): Regression Bug 815069 User impact if declined: AudioChannelAgent will be broken on nsHTMLMediaElement (b2g18 branch). So Audio Competing Policy will be malfunctional. Testing completed: Do the testing on b2g18. Risk to taking this patch (and alternatives if risky): No. String or UUID changes made by this patch: No. This problem is exposed by bug Bug 857653, and since it is tef+. This should be a tef+ too.
We need to set some flag in order for this to be considered for b2g uplift. I'll set tef?; this blocks a blocker.
blocking-b2g: --- → tef?
as per comment 6 blocks a blocker
blocking-b2g: tef? → tef+
You need to log in before you can comment on or make changes to this bug.