Page MenuHomePhabricator

[ios] Implement transcodeVideo() function on iOS
AcceptedPublic

Authored by angelika on Thu, Mar 20, 4:46 AM.
Tags
None
Referenced Files
F5059271: D14479.diff
Wed, Mar 26, 10:11 AM
F5059266: D14479.id47490.diff
Wed, Mar 26, 10:07 AM
F5056215: D14479.diff
Tue, Mar 25, 9:25 PM
F5055985: D14479.diff
Tue, Mar 25, 7:10 PM
Unknown Object (File)
Sun, Mar 23, 1:32 PM
Unknown Object (File)
Sat, Mar 22, 10:06 PM
Unknown Object (File)
Sat, Mar 22, 5:14 AM
Subscribers

Details

Reviewers
bartek
Summary

https://linear.app/comm/issue/ENG-10314/migrate-comm-app-to-use-new-media-module

Changes:

  • on the native side we check progress every 200ms and send an event to the js side. The JS side then calls the callback with the progress
  • we can specify the profile and bitrate, we can't specify fps though
  • width and height is mandatory to keep it simple (ios requires it)
  • I'm returning some basic stats (speed, size and duration) but some are missing compared to ffmpeg implementation because I don't see much sense in collecting them. However I can add them if necessary (should be easy on Android, more complicated on iOS)

I'm using AVWriter instead of AVAssetExportSession. Using AVWriter is more complicated and messy but allows for more options.

Depends on D14478

Test Plan

Send some videos on some devices and verify they're transcoded correctly

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage