-
Notifications
You must be signed in to change notification settings - Fork 47
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
WPTs need to be updated to reflect that various metrics have been removed (moved to provisional) #647
Comments
I've reviewed the tests in https://github.com/web-platform-tests/wpt/tree/70bf8f1717e4198f00a8b0aaa19af440c6f48869/webrtc-stats - 2 of them are automatically kept in sync with the IDL in the spec, and the 3 others are not impacted by the recent changes. I've filed web-platform-tests/wpt#35703 to update the MTI tests associated to the webrtc spec, for which a review would be welcomed. |
* Align WebRTC MTI stats with specs see w3c/webrtc-pc#2744 w3c/webrtc-pc#2748 w3c/webrtc-stats#647
…tonly Automatic update from web-platform-tests Align WebRTC MTI stats with specs (#35703) * Align WebRTC MTI stats with specs see w3c/webrtc-pc#2744 w3c/webrtc-pc#2748 w3c/webrtc-stats#647 -- wpt-commits: ffb6a1e86161c3c449d8220700624bbc31fb0dcd wpt-pr: 35703
@dontcallmedom should we close this issue? given the other work was merged? |
yes from my perspective, but probably worth checking if @henbos had other changes in mind to track with this issue |
Probably easiest to remove tests for unimplemented provisional metrics, in case they get implemented in the future we should add tests as part of those efforts
The text was updated successfully, but these errors were encountered: