Closed Bug 714445 Opened 8 years ago Closed 8 years ago
Web MReader::Read Metadata reads preferences off the main thread
Which is not allowed. I'm not sure if not setting mInfo.mStereoMode if the preference getter failed was intentional ... I'm going to guess it wasn't.
Attachment #585137 - Flags: review?(roc)
Attachment #585137 - Flags: review?(roc) → review+
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Comment on attachment 585137 [details] [diff] [review] Patch [Approval Request Comment] Regression caused by (bug #): Bug 584259 User impact if declined: Potentially rare crashes when using WebM <video> Testing completed (on m-c, etc.): None, but it's obvious from code inspection what's wrong here. Risk to taking this patch (and alternatives if risky): Fairly low.
Comment on attachment 585137 [details] [diff] [review] Patch [Triage Comment] Let's take this on Aurora given the low risk nature of the fix, but let's only take on beta if we find that significant crash reports (right now we think it's rare).
Depends on: 744063
You need to log in before you can comment on or make changes to this bug.