-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Test the properties of SpeechSynthesisEvent #34710
Test the properties of SpeechSynthesisEvent #34710
Conversation
This is mainly to check that they do not bubble. Follows WICG/speech-api#104.
await test_driver.bless('speechSynthesis.speak', | ||
() => speechSynthesis.speak(utterance)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just so it's clear that speak doesn't actually return anything.
await test_driver.bless('speechSynthesis.speak', | |
() => speechSynthesis.speak(utterance)); | |
await test_driver.bless('speechSynthesis.speak'); | |
speechSynthesis.speak(utterance); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The 2nd argument here is intended to be used like this, the callback for the thing that needs a user gesture. But I can wrap in {}
so that it doesn't matter what speechSynthesis.speak(utterance)
returns.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Understood. I generally don't bother using the bless' callback, as the click() resolves the bless' promise (enabling the transient activation). Just makes the code a bit easier on the eye.
But again, just a nit. All looks good 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a nit
This applies a nit from web-platform-tests#34710 and makes the "unexpected event" bits more consistent.
This applies a nit from #34710 and makes the "unexpected event" bits more consistent.
This is mainly to check that they do not bubble. Follows WICG/speech-api#104.
This applies a nit from #34710 and makes the "unexpected event" bits more consistent.
This is mainly to check that they do not bubble. Follows WICG/speech-api#104.
…m-tests#34716) This applies a nit from web-platform-tests#34710 and makes the "unexpected event" bits more consistent.
This is mainly to check that they do not bubble. Follows WICG/speech-api#104.
…m-tests#34716) This applies a nit from web-platform-tests#34710 and makes the "unexpected event" bits more consistent.
…nd tinker with others, a=testonly Automatic update from web-platform-tests Make a speech synthesis fail fast and tinker with others (#34716) This applies a nit from web-platform-tests/wpt#34710 and makes the "unexpected event" bits more consistent. -- wpt-commits: 308366a5cf759241b5a8b1a4ab11ef3865625c22 wpt-pr: 34716
This is mainly to check that they do not bubble.
Follows WICG/speech-api#104.