-
Notifications
You must be signed in to change notification settings - Fork 163
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
Emit TimesyncUpdates for local tracks #1191
base: main
Are you sure you want to change the base?
Conversation
|
size-limit report 📦
|
@@ -541,5 +543,23 @@ export default abstract class LocalTrack< | |||
this.emit(TrackEvent.TrackProcessorUpdate); | |||
} | |||
|
|||
protected registerTimeSyncUpdate() { |
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.
does this work for RN?
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.
the browser APIs requestAnimationFrame
, and performance.x
should be available in RN. Haven't done any tests yet to confirm though.
@davidliu had some concerns whether the contributing sources used for remote tracks would work in RN out of the box though
Manually computing these for local tracks makes it easier to compute
activeTranscriptions
with the same API that we use for remote tracks.Downside is that we register this for every local track independently, but probably not a major concern.