Closed Bug 838065 Opened 11 years ago Closed 11 years ago

Reenable decodeAudioData tests

Categories

(Core :: Audio/Video, defect)

18 Branch
x86_64
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla22

People

(Reporter: roc, Assigned: ehsan.akhgari)

Details

Attachments

(1 file, 1 obsolete file)

These tests are currently sensitive to the sample rate of the AudioContext. It would be nice to make them independent of the sample rate, or else at least support a variety of sample rates.
Attached patch WIP (obsolete) — Splinter Review
Currently testing on try...
Attached patch Patch (v1)Splinter Review
Attachment #719176 - Attachment is obsolete: true
Attachment #719456 - Flags: review?(roc)
Comment on attachment 719456 [details] [diff] [review]
Patch (v1)

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

Can you add something to make the tests automatically todo if the sample rate isn't 44100?
Attachment #719456 - Flags: review?(roc) → review+
Sure (assuming you mean 48000).
https://hg.mozilla.org/mozilla-central/rev/6bd0ba602b8a
Status: NEW → RESOLVED
Closed: 11 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → mozilla22
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: