Dispatch AudioSink notifications asynchronously

RESOLVED FIXED in Firefox 40

Status

()

Core
Audio/Video
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: bholley, Assigned: bholley)

Tracking

unspecified
mozilla40
x86
Mac OS X
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox40 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
I wrote patches for this in bug 1145686, but it looks like it's causing a b2g ICS emulator timeout in dom/media/test/test_loop.html that I don't have time to dig into right now. So I'm separating this patch out and landing the rest.
(Assignee)

Comment 1

3 years ago
Created attachment 8584755 [details] [diff] [review]
Dispatch all audiosink notifications asynchronously. r=jww
Attachment #8584755 - Flags: review+
(Assignee)

Comment 3

3 years ago
So it turns out that the b2g jobs that run the media tests are hidden now because they're generally failing all over the place. It's not clear to me whether the test_loop.html stuff represents a new issue or not, but I can't reliably debug it with try right now, and I need it for bug 1151656 - so I'm going to push it. Sorry in advance for any trouble it causes people, but it's pretty difficult to debug failures that only show up intermittently+b2g only.
Blocks: 1151656
(Assignee)

Comment 5

3 years ago
(In reply to Bobby Holley (:bholley) from comment #4)
> (Note - a recent green try run with this + a bunch of other patches is
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=fbff16cd9779 )

Looks like this patch wasn't actually in that push for some reason, and it had a typo. Fixing that with Ryan's permission:

https://hg.mozilla.org/integration/mozilla-inbound/rev/63c495f3e709
https://hg.mozilla.org/mozilla-central/rev/b42d9bedcbf0
https://hg.mozilla.org/mozilla-central/rev/63c495f3e709
Assignee: nobody → bobbyholley
Status: NEW → RESOLVED
Last Resolved: 3 years ago
status-firefox40: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.