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.

Sign up for GitHub

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

Jump to bottom

Add internal slots to MediaStreamTrack. #822

Merged
merged 5 commits into from Dec 2, 2021
Merged

Add internal slots to MediaStreamTrack. #822

merged 5 commits into from Dec 2, 2021

Conversation

jan-ivar
Copy link
Member

@jan-ivar jan-ivar commented Oct 13, 2021 •
edited by pr-preview bot

Fixes #812.

Preview | Diff

@jan-ivar jan-ivar self-assigned this Oct 13, 2021
@jan-ivar
Copy link
Member Author

Note this PR is on top of #821.

@jan-ivar
Copy link
Member Author

Rebased. PTAL

@jan-ivar jan-ivar requested a review from youennf October 25, 2021 15:59
alvestrand
alvestrand reviewed Nov 18, 2021
View reviewed changes
getusermedia.html Outdated Show resolved Hide resolved
@jan-ivar
Copy link
Member Author

jan-ivar commented Dec 2, 2021

Rebased on top of tip to fix errors.

@jan-ivar jan-ivar merged commit 2bd1120 into w3c:main Dec 2, 2021
@jan-ivar jan-ivar deleted the trackslots branch December 2, 2021 20:57
@jan-ivar jan-ivar mentioned this pull request Dec 2, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dontcallmedom dontcallmedom dontcallmedom left review comments

@alvestrand alvestrand alvestrand left review comments

@eladalon1983 eladalon1983 Awaiting requested review from eladalon1983

@youennf youennf Awaiting requested review from youennf

Assignees

@jan-ivar jan-ivar

Labels
editorial Editors can integrate
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Track should have an [[internal slot]] pointing to source
3 participants
@jan-ivar @dontcallmedom @alvestrand