Page MenuHomePhabricator
Differential D14480

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

Authored by angelika on Mar 20 2025, 4:47 AM.
Tags
None
Referenced Files
F6243140: D14480.id47502.diff
Fri, Apr 25, 1:20 AM
F6238430: D14480.diff
Thu, Apr 24, 6:14 PM
F6234619: D14480.diff
Thu, Apr 24, 1:27 PM
F6234561: D14480.diff
Thu, Apr 24, 1:24 PM
Unknown Object (File)
Wed, Apr 23, 11:56 PM
Unknown Object (File)
Wed, Apr 23, 2:49 PM
Unknown Object (File)
Thu, Apr 17, 1:15 PM
Unknown Object (File)
Mon, Apr 14, 8:26 PM
View All 52 Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
Commits
rCOMM085ad9283c28: [native] Use transcodeVideo() from MediaModule instead of ffmpeg
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
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

angelika created this revision.Mar 20 2025, 4:47 AM
angelika held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptMar 20 2025, 4:47 AM
angelika added a child revision: D14481: [native] Remove ffmpeg dependency.Mar 20 2025, 4:48 AM
Harbormaster completed remote builds in B33787: Diff 47491.Mar 20 2025, 5:24 AM
angelika added inline comments.Mar 20 2025, 5:38 AM
lib/types/media-types.js
107

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

We don't have return code anymore but we throw exceptions

angelika updated this revision to Diff 47502.Mar 20 2025, 5:52 AM

Fix

angelika published this revision for review.Mar 20 2025, 5:57 AM
Harbormaster completed remote builds in B33798: Diff 47502.Mar 20 2025, 6:11 AM
bartek accepted this revision.Mar 23 2025, 11:44 PM

Looks much cleaner now

This revision is now accepted and ready to land.Mar 23 2025, 11:44 PM
angelika updated this revision to Diff 47556.Mar 25 2025, 10:06 PM

Rebase

Harbormaster completed remote builds in B33838: Diff 47556.Mar 25 2025, 10:46 PM
angelika updated this revision to Diff 47592.Thu, Mar 27, 9:08 AM

Set bitrate to 4000 kb/s

Harbormaster failed remote builds in B33859: Diff 47592!Thu, Mar 27, 9:17 AM
Harbormaster completed remote builds in B33859: Diff 47592.Thu, Mar 27, 12:05 PM
Closed by commit rCOMM085ad9283c28: [native] Use transcodeVideo() from MediaModule instead of ffmpeg (authored by angelika). · Explain WhyThu, Mar 27, 12:21 PM
This revision was automatically updated to reflect the committed changes.
angelika added a commit: rCOMM085ad9283c28: [native] Use transcodeVideo() from MediaModule instead of ffmpeg.

Revision Contents
Changeset List

PathSize
lib/
media/
65 lines
types/
5 lines
native/
media/
44 lines
32 lines
utils/
51 lines

Diff 47491

View Options

lib/media/video-utils.js

  • This file was deleted.
This file was completely deleted. Show File Contents
View Options

lib/types/media-types.js

  • This file was deleted.
This file was completely deleted. Show File Contents
View Options

native/media/ffmpeg.js

  • This file was deleted.
This file was completely deleted. Show File Contents
View Options

native/media/video-utils.js

  • This file was deleted.
This file was completely deleted. Show File Contents
View Options

native/utils/media-module.js

Loading...