Open Bug 1573234 Opened 5 years ago Updated 5 years ago

resume() method should be on AudioContext and OfflineAudioContext, not on BaseAudioContext

Categories

(Core :: Web Audio, defect, P3)

defect

Tracking

()

People

(Reporter: bzbarsky, Assigned: padenot)

References

Details

It's on the separate interfaces in the spec, not on the base interface.

It looks to me like other browsers are following the spec here. Is there a reason we have the different behavior?

Note also bug 1265406. If we fix this bug, we might want to actually not define resume() on OfflineAudioContext until we actually support it.

Marking this a P3. @jib, is something that you'd look at? Please re-route, if you know of someone better suited.

Flags: needinfo?(jib)
Priority: -- → P3

(In reply to Boris Zbarsky [:bzbarsky, bz on IRC] from comment #0)

Is there a reason we have the different behavior?

Looks like this is hangover from when OfflineAudioContext inherited AudioContext, but it was the spec'd behavior for quite some time. https://github.com/WebAudio/web-audio-api/issues/1060

https://github.com/WebAudio/web-audio-api/issues/1669#issuecomment-399169133 separated the methods, which probably would have been better from the start anyway.

Note also bug 1265406. If we fix this bug, we might want to actually not define resume() on OfflineAudioContext until we actually support it.

Agreed.

Flags: needinfo?(jib) → needinfo?(padenot)
Assignee: nobody → padenot
Flags: needinfo?(padenot)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.