1324352 - Implement BaseAudioContext

Status: RESOLVED FIXED

(Core :: Web Audio, defect, P2)

Description

[Andrea Marchesini [:baku]]

The spec has BaseAudioContext and AudioContext interface. Our implementation uses just AudioContext.

https://webaudio.github.io/web-audio-api/#idl-def-BaseAudioContext

Comment 1

[Andrea Marchesini [:baku]]

Attachment: audioContext.patch

Comment 2

[Paul Adenot (:padenot)]

Comment on attachment 8820026 [details] [diff] [review]
audioContext.patch

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

::: dom/webidl/BaseAudioContext.webidl
@@ +60,5 @@
> +    [NewObject, Throws]
> +    GainNode createGain();
> +
> +    [NewObject, Throws]
> +    DelayNode createDelay(optional double maxDelayTime = 1); // TODO: no = 1

This just got merged in the spec, it was a mistake to remove it, so you can remove this comment.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/50a74758ee7d
Implement BaseAudioContext, r=padenot

Comment 4

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a3a17725724
Backed out changeset 50a74758ee7d for failing webaudioeditor devtools tests like test/browser_audionode-actor-bypass.js. r=backout

Comment 5

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

Backed out for failing webaudioeditor devtools tests like test/browser_audionode-actor-bypass.js:

Bug 1324352:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9a3a17725724d71bd5906d44018da99e42e9cad8

Bug 1324568:
https://hg.mozilla.org/integration/mozilla-inbound/rev/4ee212f486d788dc8343605f9e364b654834854b
https://hg.mozilla.org/integration/mozilla-inbound/rev/bf74c90bff441b33a29e2a15a41e83b551851dc7

Bug 1324659:
https://hg.mozilla.org/integration/mozilla-inbound/rev/faf170d0bdba2f09f6b5875099a930109591fd14

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=f350bbf9e6363a685520b2ad9ba8071f6eebfd07
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=40827534&repo=mozilla-inbound

[task 2016-12-20T20:13:00.006985Z] 20:13:00     INFO - [1627] WARNING: Audio Buffer is not full by the end of the callback.: 'Available() == 0 || mSampleWriteOffset == 0', file /home/worker/workspace/build/src/dom/media/AudioBufferUtils.h, line 88
[task 2016-12-20T20:13:01.277772Z] 20:13:01     INFO - ++DOCSHELL 0x7f67fb30b000 == 10 [pid = 1627] [id = {f9f7e02a-19af-475c-9af7-85c14d9a869b}]
[task 2016-12-20T20:13:01.277890Z] 20:13:01     INFO - ++DOMWINDOW == 25 (0x7f67fb310000) [pid = 1627] [serial = 25] [outer = (nil)]
[task 2016-12-20T20:13:01.279748Z] 20:13:01     INFO - ++DOMWINDOW == 26 (0x7f67fb31d000) [pid = 1627] [serial = 26] [outer = 0x7f67fb310000]
[task 2016-12-20T20:13:01.295432Z] 20:13:01     INFO - ++DOMWINDOW == 27 (0x7f6800f6b000) [pid = 1627] [serial = 27] [outer = 0x7f67fb310000]
[task 2016-12-20T20:13:01.320296Z] 20:13:01     INFO - ++DOCSHELL 0x7f6800f58800 == 11 [pid = 1627] [id = {ee1c0fa0-4b5f-43f1-95fc-0637e5e50c3f}]
[task 2016-12-20T20:13:01.321263Z] 20:13:01     INFO - ++DOMWINDOW == 28 (0x7f6802286000) [pid = 1627] [serial = 28] [outer = (nil)]
[task 2016-12-20T20:13:01.356084Z] 20:13:01     INFO - ++DOMWINDOW == 29 (0x7f67fb147800) [pid = 1627] [serial = 29] [outer = 0x7f6802286000]
[task 2016-12-20T20:13:04.683547Z] 20:13:04     INFO - --DOCSHELL 0x7f680a983000 == 10 [pid = 1627] [id = {45d20fa8-0ac8-4107-97e5-ee669183ff49}]
[task 2016-12-20T20:13:15.017220Z] 20:13:15     INFO - --DOMWINDOW == 28 (0x7f68025dd000) [pid = 1627] [serial = 19] [outer = (nil)] [url = about:blank]
[task 2016-12-20T20:13:15.017371Z] 20:13:15     INFO - --DOMWINDOW == 27 (0x7f6800fd4000) [pid = 1627] [serial = 21] [outer = (nil)] [url = about:blank]
[task 2016-12-20T20:13:15.017450Z] 20:13:15     INFO - --DOMWINDOW == 26 (0x7f680a539800) [pid = 1627] [serial = 9] [outer = (nil)] [url = about:blank]
[task 2016-12-20T20:13:15.017543Z] 20:13:15     INFO - --DOMWINDOW == 25 (0x7f6818f16000) [pid = 1627] [serial = 2] [outer = (nil)] [url = about:blank]
[task 2016-12-20T20:13:15.018964Z] 20:13:15     INFO - --DOMWINDOW == 24 (0x7f68163da800) [pid = 1627] [serial = 23] [outer = (nil)] [url = http://example.com/browser/devtools/client/webaudioeditor/test/doc_simple-context.html]
[task 2016-12-20T20:13:15.019051Z] 20:13:15     INFO - --DOMWINDOW == 23 (0x7f67fb31d000) [pid = 1627] [serial = 26] [outer = (nil)] [url = about:blank]
[task 2016-12-20T20:13:16.826351Z] 20:13:16     INFO - --DOMWINDOW == 22 (0x7f680a983800) [pid = 1627] [serial = 8] [outer = (nil)] [url = about:blank]
[task 2016-12-20T20:13:21.946723Z] 20:13:21     INFO - --DOMWINDOW == 21 (0x7f6805e9d000) [pid = 1627] [serial = 13] [outer = (nil)] [url = about:blank]
[task 2016-12-20T20:13:21.947272Z] 20:13:21     INFO - --DOMWINDOW == 20 (0x7f680a355000) [pid = 1627] [serial = 12] [outer = (nil)] [url = about:blank]
[task 2016-12-20T20:13:43.803055Z] 20:13:43     INFO - TEST-INFO | started process screentopng
[task 2016-12-20T20:13:44.296904Z] 20:13:44     INFO - TEST-INFO | screentopng: exit 0
[task 2016-12-20T20:13:44.297150Z] 20:13:44     INFO - Buffered messages logged at 20:12:58
[task 2016-12-20T20:13:44.297309Z] 20:13:44     INFO - Entering test bound 
[task 2016-12-20T20:13:44.297993Z] 20:13:44     INFO - Initializing a web audio editor front.
[task 2016-12-20T20:13:44.300684Z] 20:13:44     INFO - Adding tab: http://example.com/browser/devtools/client/webaudioeditor/test/doc_simple-context.html
[task 2016-12-20T20:13:44.300801Z] 20:13:44     INFO - Buffered messages logged at 20:12:59
[task 2016-12-20T20:13:44.301309Z] 20:13:44     INFO - Tab added and finished loading: http://example.com/browser/devtools/client/webaudioeditor/test/doc_simple-context.html
[task 2016-12-20T20:13:44.301373Z] 20:13:44     INFO - Waiting for 3 create-node events
[task 2016-12-20T20:13:44.302286Z] 20:13:44     INFO - Buffered messages finished
[task 2016-12-20T20:13:44.302440Z] 20:13:44     INFO - TEST-UNEXPECTED-FAIL | devtools/client/webaudioeditor/test/browser_audionode-actor-bypass.js | Test timed out - 
[task 2016-12-20T20:13:44.302554Z] 20:13:44     INFO - finish() was called, cleaning up...
[task 2016-12-20T20:13:44.435638Z] 20:13:44     INFO - MEMORY STAT vsizeMaxContiguous not supported in this build configuration.
[task 2016-12-20T20:13:44.435880Z] 20:13:44     INFO - MEMORY STAT | vsize 1131MB | residentFast 251MB | heapAllocated 88MB
[task 2016-12-20T20:13:44.436072Z] 20:13:44     INFO - TEST-OK | devtools/client/webaudioeditor/test/browser_audionode-actor-bypass.js | took 45972ms
[task 2016-12-20T20:13:44.437750Z] 20:13:44     INFO - Not taking screenshot here: see the one that was previously logged
[task 2016-12-20T20:13:44.439613Z] 20:13:44     INFO - TEST-UNEXPECTED-FAIL | devtools/client/webaudioeditor/test/browser_audionode-actor-bypass.js | Found a tab after previous test timed out: http://example.com/browser/devtools/client/webaudioeditor/test/doc_simple-context.html -

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/6823579dd938
Implement BaseAudioContext, r=padenot

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6823579dd938

Comment 8

[Chris Mills [:cmills]]

This has been documented.