1209598 - suspend() fails when AudioContext.state is "suspended"

Status: RESOLVED DUPLICATE of bug 1269741

(Core :: Web Audio, defect, P2)

Description

[Karl Tomlinson (:karlt)]

It should be possible to suspend() a new AudioContext or an AudioContext on which resume() has recently been called, but state has not yet changed.

Comment 1

[Karl Tomlinson (:karlt)]

Noticed in bug 1164814 comment 1 and bug 1198386 comment 572.

Comment 2

[(no longer active)]

It's sad that the spec is completely silent about what actually needs to happen in suspend().  <http://webaudio.github.io/web-audio-api/#widl-AudioContext-suspend-Promise-void>  We should fix the spec as well.

Comment 3

[Paul Adenot (:padenot)]

(In reply to Ehsan Akhgari (don't ask for review please) from comment #2)
> It's sad that the spec is completely silent about what actually needs to
> happen in suspend(). 
> <http://webaudio.github.io/web-audio-api/#widl-AudioContext-suspend-Promise-
> void>  We should fix the spec as well.

I'm currently speccing the processing model of the Web Audio API (to have a solid ground on which we can build what was called the AudioWorker), and in particular asynchronous and synchronous parts [0] of method calls, this will allow us to spec this properly.

I was going to go all over the spec and mark synchronous and asynchronous parts of all the operations today, so I can do that as I fix everything else, as well. This is due to merge hopefully before TPAC or shortly after.

[0]: http://padenot.github.io/web-audio-api/#asynchronous-operations

Comment 4

[Paul Adenot (:padenot)]

(In reply to Paul Adenot (:padenot) from comment #3)
> (In reply to Ehsan Akhgari (don't ask for review please) from comment #2)
> > It's sad that the spec is completely silent about what actually needs to
> > happen in suspend(). 
> > <http://webaudio.github.io/web-audio-api/#widl-AudioContext-suspend-Promise-
> > void>  We should fix the spec as well.
> 
> I'm currently speccing the processing model of the Web Audio API (to have a
> solid ground on which we can build what was called the AudioWorker), and in
> particular asynchronous and synchronous parts [0] of method calls, this will
> allow us to spec this properly.
> 
> I was going to go all over the spec and mark synchronous and asynchronous
> parts of all the operations today, so I can do that as I fix everything
> else, as well. This is due to merge hopefully before TPAC or shortly after.
> 
> [0]: http://padenot.github.io/web-audio-api/#asynchronous-operations

This is now done, still on my branch, and up for review to the group. For example, close:

http://padenot.github.io/web-audio-api/#widl-AudioContext-close-Promise-void

I'll now fix gecko.

Comment 5

[(no longer active)]

Thanks, Paul!