Open
Bug 1424271
Opened 7 years ago
Updated 2 years ago
Move some more Web Audio API tests from mochitest to web-platform-tests
Categories
(Core :: Web Audio, enhancement, P3)
Tracking
()
NEW
People
(Reporter: alan.guyoch, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [need info padenot 2017-12-12])
Attachments
(12 files)
(deleted),
text/x-review-board-request
|
padenot
:
review-
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
Details | |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review-
|
Details |
(deleted),
text/x-review-board-request
|
padenot
:
review+
|
Details |
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:57.0) Gecko/20100101 Firefox/57.0
Build ID: 20171130135210
Comment 1•7 years ago
|
||
Paul, can you please handle this one?
Flags: needinfo?(padenot)
Whiteboard: [need info padenot 2017-12-12]
Updated•7 years ago
|
Assignee: nobody → alan.guyoch
Status: UNCONFIRMED → ASSIGNED
Rank: 25
Ever confirmed: true
Flags: needinfo?(padenot)
Priority: -- → P3
Summary: Move some Web Audio API tests from mochitest to web-platform-tests → Move some more Web Audio API tests from mochitest to web-platform-tests
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 14•7 years ago
|
||
mozreview-review |
Comment on attachment 8940614 [details]
Bug 1424271 - convert mochitest test_audioBufferSourceNodeLoopStartEnd.html to a web-platform-test
https://reviewboard.mozilla.org/r/210834/#review216642
Attachment #8940614 -
Flags: review?(padenot) → review+
Comment 15•7 years ago
|
||
mozreview-review |
Comment on attachment 8940614 [details]
Bug 1424271 - convert mochitest test_audioBufferSourceNodeLoopStartEnd.html to a web-platform-test
https://reviewboard.mozilla.org/r/210834/#review216646
Comment 16•7 years ago
|
||
mozreview-review |
Comment on attachment 8940619 [details]
Bug 1424271 - convert mochitest test_AudioListener.html to a web-platform-test
https://reviewboard.mozilla.org/r/210844/#review216640
::: commit-message-ccbff:1
(Diff revision 1)
> +Bug 1424271 - convert mochitest test_AudioListener.html to a web-platform-test r?padenot
No need for this one, it's deprecated, and has been removed from the spec.
Attachment #8940619 -
Flags: review?(padenot) → review-
Comment 17•7 years ago
|
||
mozreview-review |
Comment on attachment 8940619 [details]
Bug 1424271 - convert mochitest test_AudioListener.html to a web-platform-test
https://reviewboard.mozilla.org/r/210844/#review216648
Comment 18•7 years ago
|
||
mozreview-review |
Comment on attachment 8940610 [details]
Bug 1424271 - convert mochitest test_AudioBuffer.html to a web-platform-test
https://reviewboard.mozilla.org/r/210826/#review216638
::: testing/web-platform/tests/webaudio/the-audio-api/the-audiobuffer-interface/test_AudioBuffer.html:11
(Diff revision 1)
> + <script src="/resources/testharnessreport.js"></script>
> + <script>
> +var t = async_test();
> +var context = new AudioContext();
> +var buffer = context.createBuffer(2, 2048, context.sampleRate);
> +//SpecialPowers.gc(); // Make sure that our channels are accessible after GC
Remove this, we don't want comments here.
Attachment #8940610 -
Flags: review?(padenot) → review+
Comment 19•7 years ago
|
||
mozreview-review |
Comment on attachment 8940609 [details]
Bug 1424271 - convert mochitest test_analyserNodePassThrough.html and test_analyserNodePassThrough.html to a web-platform-test
https://reviewboard.mozilla.org/r/210824/#review216730
This uses an API that is proprietary and that we use for the developers tools, we won't be importing this one.
Attachment #8940609 -
Flags: review?(padenot) → review-
Comment 20•7 years ago
|
||
mozreview-review |
Comment on attachment 8940611 [details]
Bug 1424271 - convert mochitest test_audioBufferSourceNodeEnded.html to a web-platform-test
https://reviewboard.mozilla.org/r/210828/#review216732
Attachment #8940611 -
Flags: review?(padenot) → review+
Comment 21•7 years ago
|
||
mozreview-review |
Comment on attachment 8940612 [details]
Bug 1424271 - convert mochitest test_audioBufferSourceNodeLazyLoopParam.html to a web-platform-test
https://reviewboard.mozilla.org/r/210830/#review216734
Attachment #8940612 -
Flags: review?(padenot) → review+
Comment 22•7 years ago
|
||
mozreview-review |
Comment on attachment 8940613 [details]
Bug 1424271 - convert mochitest test_audioBufferSourceNodeLoop.html to a web-platform-test
https://reviewboard.mozilla.org/r/210832/#review216736
Attachment #8940613 -
Flags: review?(padenot) → review+
Comment 23•7 years ago
|
||
mozreview-review |
Comment on attachment 8940615 [details]
Bug 1424271 - convert mochitest test_audioBufferSourceNodeLoopStartEndSame.html to a web-platform-test
https://reviewboard.mozilla.org/r/210836/#review216738
Attachment #8940615 -
Flags: review?(padenot) → review+
Comment 24•7 years ago
|
||
mozreview-review |
Comment on attachment 8940616 [details]
Bug 1424271 - convert mochitest test_audioBufferSourceNodeOffset.html to a web-platform-test
https://reviewboard.mozilla.org/r/210838/#review216740
::: testing/web-platform/tests/webaudio/the-audio-api/the-audiobuffersourcenode-interface/test_audioBufferSourceNodeOffset.html:13
(Diff revision 1)
> +var fuzz = 0.3;
> +
> +if (navigator.platform.startsWith("Mac")) {
> + // bug 895720
> + fuzz = 0.6;
> +}
Not sure I like this too much. We should try pushing it without.
Attachment #8940616 -
Flags: review?(padenot)
Comment 25•7 years ago
|
||
mozreview-review |
Comment on attachment 8940617 [details]
Bug 1424271 - convert mochitest test_AudioContext.html to a web-platform-test
https://reviewboard.mozilla.org/r/210840/#review216742
Attachment #8940617 -
Flags: review?(padenot) → review+
Comment 26•7 years ago
|
||
mozreview-review |
Comment on attachment 8940618 [details]
Bug 1424271 - convert mochitest test_audioDestinationNode.html to a web-platform-test
https://reviewboard.mozilla.org/r/210842/#review216744
Attachment #8940618 -
Flags: review?(padenot) → review+
Comment 27•7 years ago
|
||
mozreview-review |
Comment on attachment 8940620 [details]
Bug 1424271 - convert mochitest test_audioParamChaining.html to a web-platform-test
https://reviewboard.mozilla.org/r/210846/#review216748
Attachment #8940620 -
Flags: review?(padenot) → review+
Updated•5 years ago
|
Blocks: webaudio-wpt
Comment 28•3 years ago
|
||
The bug assignee didn't login in Bugzilla in the last 7 months.
:padenot, could you have a look please?
For more information, please visit auto_nag documentation.
Assignee: alan.guyoch → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(padenot)
Updated•3 years ago
|
Flags: needinfo?(padenot)
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•