Implement MediaElementAudioSourceNode

RESOLVED FIXED in Firefox 25

Status

()

defect
RESOLVED FIXED
6 years ago
5 years ago

People

(Reporter: kinetik, Assigned: roc)

Tracking

({dev-doc-complete})

Trunk
mozilla26
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(firefox25 fixed, firefox26 fixed)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment, 2 obsolete attachments)

Reporter

Updated

6 years ago
Blocks: 855570

Comment 1

6 years ago
Matthew, did you want to work on this?  :-)

FWIW I think we should first implement MediaStreamAudioSourceNode (bug 856361) and then use mozCaptureStream to implement this node type.
Depends on: 856361
Reporter

Comment 2

6 years ago
I don't think I've got time right now sorry, I just wanted to file it for the bug 855570 deps.

Updated

6 years ago
Assignee: nobody → ehsan

Comment 3

6 years ago
Mass moving Web Audio bugs to the Web Audio component.  Filter on duckityduck.
Component: Video/Audio → Web Audio

Comment 5

6 years ago
Posted patch WIP (obsolete) — Splinter Review

Comment 6

6 years ago
Posted patch WIP (obsolete) — Splinter Review
Added a test, it doesn't pass and it crashes.  But Bobby wanted to see it.
Attachment #761214 - Attachment is obsolete: true
Assignee: ehsan → roc
The test originally attached to this patch won't work since playback of the media element may introduce silence (especially at the start) while the media element decoding pipeline spins up.
Posted patch patchSplinter Review
Basically you already had, which I've reviewed/tweaked a bit, plus a new test which is just test_mediaStreamAudioSourceNodeResampling.html but using MediaElementAudioSourceNode instead of MediaStreamAudioSourceNode. Nice and simple implementation :-).
Attachment #761761 - Attachment is obsolete: true
Attachment #780774 - Flags: review?(ehsan)

Comment 9

6 years ago
Comment on attachment 780774 [details] [diff] [review]
patch

Review of attachment 780774 [details] [diff] [review]:
-----------------------------------------------------------------

It would also be really great if you could add a test similar to test_mediaStreamAudioSourceNode.html.  r=me with that.
Attachment #780774 - Flags: review?(ehsan) → review+
You mean where the track sample rate is equal to the context sample rate? I don't think that's really necessary given this is a very thin wrapper around MediaStreamAudioSourceNode.

Comment 11

6 years ago
(In reply to Robert O'Callahan (:roc) (Mozilla Corporation) from comment #10)
> You mean where the track sample rate is equal to the context sample rate? I

Yes.

> don't think that's really necessary given this is a very thin wrapper around
> MediaStreamAudioSourceNode.

Hmm that's a fair point given our knowledge about the implementation, but I'd still like us to test the API surface there.  If you want, I'm fine with us doing this in another bug though.
https://hg.mozilla.org/mozilla-central/rev/27f384522675
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla26

Comment 16

6 years ago
Comment on attachment 780774 [details] [diff] [review]
patch

This is needed for Web Audio in Firefox 25.
Attachment #780774 - Flags: approval-mozilla-aurora?

Comment 17

6 years ago
checkin-needed for Aurora, a=webaudio.
Keywords: checkin-needed

Updated

6 years ago
Attachment #780774 - Flags: approval-mozilla-aurora?
Assuming this doesn't need QA.
Whiteboard: [qa-]
Depends on: 934100
Depends on: 937718
Depends on: 937962
Depends on: 944924
You need to log in before you can comment on or make changes to this bug.