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

outputLatency normative, non-optional #2330

Closed
svgeesus opened this issue Apr 30, 2021 · 4 comments
Closed

outputLatency normative, non-optional #2330

svgeesus opened this issue Apr 30, 2021 · 4 comments

Comments

@svgeesus
Copy link
Contributor

Describe the issue

outputLatency is a required part of the spec.

Where Is It

https://webaudio.github.io/web-audio-api/#dom-audiocontext-outputlatency

Additional Information

We don't have two implementations. The Chrome bug indicates a lack of urgency to add this.

Given that, I was asked on today's transition call why is is not optional, or non-normative. Is it not implemented because of fingerprinting risk (even if quantized) or because of difficulty in estimating the latency?

@svgeesus svgeesus added Needs Discussion The issue needs more discussion before it can be fixed. Untriaged labels Apr 30, 2021
@rtoy
Copy link
Member

rtoy commented Apr 30, 2021

That's a good question. Unless we can resolve that here in this issue, we might have to wait for the teleconf next week to decide on what to do.

But there is probably a fingerprinting risk.

I also think estimating the latency is difficult and sometimes devices lie. I defer to @padenot about that since Mozilla has implemented this.

@svgeesus
Copy link
Contributor Author

To be clear, this one isn't a blocker on publishing a Proposed Rec; the transition was approved today.

The fingerprinting risk is already mentioned in the spec.

@rtoy
Copy link
Member

rtoy commented May 6, 2021

Teleconf: outputLatency is very important for synchronizing audio and video together. MediaElement does this automatically, so the appropriate code is in each browser.

So, there's no change needed here. This is required.

@rtoy rtoy removed Needs Discussion The issue needs more discussion before it can be fixed. Untriaged labels May 6, 2021
@rtoy
Copy link
Member

rtoy commented May 11, 2021

Closing. This is not an optional part of the spec. It is a bug in Chrome (and Safari?) that it is not yet implemented.

@rtoy rtoy closed this as completed May 11, 2021
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

2 participants