-
Notifications
You must be signed in to change notification settings - Fork 1.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
feat: Add KeyStatusChanged and UpdateState events #5695
Conversation
Incremental code coverage: No instrumented code was changed. |
Some tests fail but I think it is not because of this PR. @joeyparrish can you review it? (see: https://github.com/shaka-project/shaka-player/actions/runs/6350193501/attempts/6) |
lib/player.js
Outdated
* @event shaka.Player.UpdateState | ||
* @description Fired when player state is updated. | ||
* @property {string} type | ||
* 'updatestate' |
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.
It is more conventional to call a change event something like "statechanged", as you did for "keystatuschanged".
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.
Done!
It is suspicious that Chrome & Edge both failed to start on every platform. It's certainly not caused by your changes. Probably something happened on GitHub's VM images or in Chrome's releases to trigger this. |
Closes #1365