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

WPTs need to be updated to reflect that various metrics have been removed (moved to provisional) #647

Open
henbos opened this issue Jun 30, 2022 · 3 comments

Comments

@henbos
Copy link
Collaborator

henbos commented Jun 30, 2022

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

@dontcallmedom
Copy link
Member

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.

dontcallmedom added a commit to web-platform-tests/wpt that referenced this issue Aug 30, 2022
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue Sep 5, 2022
…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
@vr000m
Copy link
Contributor

vr000m commented Oct 4, 2022

@dontcallmedom should we close this issue? given the other work was merged?

@dontcallmedom
Copy link
Member

yes from my perspective, but probably worth checking if @henbos had other changes in mind to track with this issue

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

No branches or pull requests

3 participants