This leads to assertions in b2g debug builds.
Assignee: nobody → mrbkap
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: 7 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.