Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TAG review of Web Audio API (round two) #212

Closed
3 of 5 tasks
svgeesus opened this issue Nov 7, 2017 · 3 comments
Closed
3 of 5 tasks

TAG review of Web Audio API (round two) #212

svgeesus opened this issue Nov 7, 2017 · 3 comments
Assignees

Comments

@svgeesus
Copy link

svgeesus commented Nov 7, 2017

Dear Sirs and Madam of TAG!

I'm requesting a TAG review of:

Further details (optional):

You should also know that...

The previous TAG review of this specification is still relevant; please let us know if we failed to address any point in that review. Domenic continued to provide guidance on issues raised from that earlier review. So this is a request for a second round of review to cover more recent developments and ensure we got everything right from the first round.

The major change since that earlier review is the extensibility point. We switched from ScriptProcessorNode (running on the main thread, now deprecated) to Audio worklets which are of course derived from the TAG/CSS Houdini Worklet. Plus a lot of detailed spec improvements and clarifications. Complete changes list.

Tests are being ported into WPT from the Mozilla and Chrome tests; this only started a couple of weeks ago but will fill up in the coming weeks.

We'd prefer the TAG provide feedback as (please select one):

  • open issues in our Github repo for each point of feedback, using the TAG label
  • open a single issue in our Github repo for the entire review
  • leave review feedback as a comment in this issue and @-notify [github usernames]

Please preview the issue and check that the links work before submitting

For background, some decent explainers:

https://github.com/w3c/ServiceWorker/blob/master/explainer.md
https://github.com/zkoch/paymentrequest/blob/gh-pages/docs/explainer.md
https://github.com/WICG/IntersectionObserver/blob/gh-pages/explainer.md (although this one includes IDL, which an explainer should not)

@cynthia cynthia self-assigned this Nov 29, 2017
@plinss plinss added this to the tag-telcon-2018-01-30 milestone Dec 5, 2017
@dbaron
Copy link
Member

dbaron commented Feb 2, 2018

WebAudio/web-audio-api#1471 was raised as worth discussing

@torgo
Copy link
Member

torgo commented Feb 2, 2018

@slightlyoff to leave some feedback on their WebAudio/web-audio-api#1471.

@torgo
Copy link
Member

torgo commented Feb 2, 2018

We discussed with chairs at london f2f. We are happy with current state of things. Thanks for flying TAG.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants