Differential D14480

[native] Use transcodeVideo() from MediaModule instead of ffmpeg
AcceptedPublic
Actions

Authored by angelika on Thu, Mar 20, 4:47 AM.

Tags

None

Referenced Files

	F5024746: D14480.id.diff
	Mon, Mar 24, 1:14 AM

	F4994103: D14480.diff
	Sun, Mar 23, 5:01 PM

	F4974343: D14480.diff
	Sun, Mar 23, 1:35 PM

Subscribers

Details

Reviewers

Summary

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

Depends on D14479

Test Plan

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

Diff Detail

Repository

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

angelika created this revision.Thu, Mar 20, 4:47 AM

angelika held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptThu, Mar 20, 4:47 AM

angelika added a child revision: D14481: [native] Remove ffmpeg dependency.Thu, Mar 20, 4:48 AM

Harbormaster completed remote builds in B33787: Diff 47491.Thu, Mar 20, 5:24 AM

angelika added inline comments.Thu, Mar 20, 5:38 AM

lib/types/media-types.js
107 ↗	(On Diff #47491)	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)
118 ↗	(On Diff #47491)	We don't have return code anymore but we throw exceptions

Fix

angelika published this revision for review.Thu, Mar 20, 5:57 AM

Harbormaster completed remote builds in B33798: Diff 47502.Thu, Mar 20, 6:11 AM

Looks much cleaner now

This revision is now accepted and ready to land.Sun, Mar 23, 11:44 PM

Revision Contents
Changeset List

Path

Size

lib/

media/

65 lines

types/

5 lines

native/

media/

44 lines

32 lines

utils/

media-module.js

51 lines

Diff 47502

lib/media/video-utils.js

Loading...

lib/types/media-types.js

Loading...

native/media/ffmpeg.js

Loading...

native/media/video-utils.js

Loading...

native/utils/media-module.js

Loading...