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

Test Depth Stream with MediaStream interface #7457

Closed

Conversation

xiuqijix
Copy link
Contributor

@xiuqijix xiuqijix commented Sep 25, 2017

Reuse tests from https://github.com/w3c/web-platform-tests/blob/master/mediacapture-streams/
to test Depth Stream with some optimizations:

  • Move proper tests into one file to reduce testing effort
  • Rewrite most tests with promise_test

@wpt-pr-bot
Copy link
Collaborator

There are no owners for this pull request. Please reach out on W3C's irc server (irc.w3.org, port 6665) on channel #testing (web client) to get help with this. Thank you!

@ghost
Copy link

ghost commented Sep 25, 2017

Build PASSED

Started: 2017-09-25 02:43:56
Finished: 2017-09-25 02:49:08

View more information about this build on:

 Reuse tests from https://github.com/w3c/web-platform-tests/blob/master/mediacapture-streams/
 to test Depth Stream with some optimizations:
  - Move proper tests into one file to reduce testing effort
  - Rewrite most tests with `promise_test`
@foolip
Copy link
Member

foolip commented Oct 5, 2017

@kaixinjxq, can you add an OWNERS file given that this is a new directory? Add whoever you think would be a good review and see if they complain :)

@xiuqijix
Copy link
Contributor Author

This PR is blocked on the required "Travis CI - Pull Request" check after #14499. In order to trigger it, I will close and reopen this PR.

@anssiko
Copy link
Member

anssiko commented Oct 31, 2022

We retired this spec and deprecated the implementation. Anyway, thanks for this contribution!

@anssiko anssiko closed this Oct 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants