Page MenuHomePhabricator
Differential D14474

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

Authored by angelika on Mar 20 2025, 4:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 23, 3:15 AM
Unknown Object (File)
Thu, Apr 17, 3:08 PM
Unknown Object (File)
Wed, Apr 16, 7:54 PM
Unknown Object (File)
Sat, Apr 12, 7:04 PM
Unknown Object (File)
Sat, Apr 12, 1:20 PM
Unknown Object (File)
Wed, Apr 9, 9:10 PM
Unknown Object (File)
Wed, Apr 9, 4:07 PM
Unknown Object (File)
Wed, Apr 9, 2:59 AM
View All 31 Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
Commits
rCOMMa21fb87bf067: [native] Use hasMultipleFrames() from MediaModule instead of ffmpeg
Summary

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

Depends on D14473

Test Plan

Try to send a gif with animation and without animation and verify that the function returns correct result.
Set mediaType to 'photo_or_video' for gifs in order to trigger the check.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
lib/
media/
18 lines
native/
media/
20 lines
utils/
5 lines

Diff 47550

View Options

lib/media/video-utils.js

Loading...
View Options

native/media/ffmpeg.js

Loading...
View Options

native/utils/media-module.js

Loading...