Page MenuHomePhabricator
Differential D7475

[native] Move definition of `onTranscodingProgress` outside of `mediaProcessConfig`
AbandonedPublicDraft
Actions

Authored by atul on Apr 17 2023, 1:18 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 9, 8:36 PM
Unknown Object (File)
Fri, Nov 8, 9:22 PM
Unknown Object (File)
Thu, Nov 7, 12:42 PM
Unknown Object (File)
Thu, Nov 7, 10:36 AM
Unknown Object (File)
Thu, Nov 7, 10:32 AM
Unknown Object (File)
Thu, Nov 7, 8:55 AM
Unknown Object (File)
Thu, Nov 7, 8:39 AM
Unknown Object (File)
Tue, Nov 5, 2:10 AM
View All 36 Files
Subscribers
ashoat
tomek

Details

Reviewers
None
Summary

It doesn't make sense to define onTranscodingProgress inside of mediaProcessConfig. Instead it makes sense to define onTranscodingProgress callback at callsite and pass into mediaProcessConfig. This also makes mediaProcessConfig more generally useful.

I initially added the onTranscodingProgress stuff to InputStateContainer so my bad.

(I'm NOT planning on doing any more refactoring here. This was confusing and 2 min change so I just made it.)

Test Plan

Still able to send video message and the transcoding progress shows up as expected.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
native/
input/
13 lines
media/
2 lines

Diff 25233

View Options

native/input/input-state-container.react.js

Loading...
View Options

native/media/media-utils.js

Loading...