1425277 - Have a unified MediaDataEncoder API

Status: RESOLVED FIXED

(Core :: WebRTC: Audio/Video, enhancement, P3)

Description

[Jean-Yves Avenard [:jya]]

We do not have a unified encoders API like we do with decoders.

We want an asynchronous, promise based encoding API similar to the MediaDataDecoder API

This would be used with WebRTC (as well as MediaRecoder) to support hardware encoding (with software fallback)

Comment 1

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1425277 - p1: introduce media encoder interfaces. r?jya

Comment 2

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1425277 - p2: remove dynamic apple framework linkers. r?jya No need for own linkers anymore for CoreMedia and VideoToolbox are both available on our minimal supported version (10.9).

No need for own linkers anymore for CoreMedia and VideoToolbox are both available
on our minimal supported version (10.9).

Depends on D7557

Comment 3

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1425277 - p3: fix misplaced dependent header. r?jya To address compile error when AppleVTDeocder.cpp is moved to another unified source file or in non-unified builds.

To address compile error when AppleVTDeocder.cpp is moved to another unified source file or in non-unified builds.

Depends on D7558

Comment 4

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1425277 - p4: RAII for CVBuffer. r?jya

Depends on D7559

Comment 5

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1425277 - p5: add Append() to MediaRawDataWriter. r?jya

Depends on D7560

Comment 6

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1425277 - p6: implement media encoder using Apple VideoToolbox API. r?jya

Depends on D7561

Comment 7

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1425277 - p7: add apple encoder module and factory. r?jya

Depends on D7562

Comment 8

[John Lin [:jhlin][:jolin]]

Attachment: Bug 1425277 - p8: add gtest cases for media encoder. r?jya

Depends on D7563

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/725677ef4b8e
p1: introduce media encoder interfaces. r=jya
https://hg.mozilla.org/integration/autoland/rev/d1255f131fe4
p2: remove dynamic apple framework linkers. r=jya No need for own linkers anymore for CoreMedia and VideoToolbox are both available on our minimal supported version (10.9).
https://hg.mozilla.org/integration/autoland/rev/4dd46901846a
p3: fix misplaced dependent header. r=jya To address compile error when AppleVTDeocder.cpp is moved to another unified source file or in non-unified builds.
https://hg.mozilla.org/integration/autoland/rev/6dbd2767e7ab
p4: RAII for CVBuffer. r=jya
https://hg.mozilla.org/integration/autoland/rev/3bf7f358af16
p5: add Append() to MediaRawDataWriter. r=jya
https://hg.mozilla.org/integration/autoland/rev/b86195e2307a
p6: implement media encoder using Apple VideoToolbox API. r=jya
https://hg.mozilla.org/integration/autoland/rev/8759840bbdd3
p7: add apple encoder module and factory. r=jya
https://hg.mozilla.org/integration/autoland/rev/74bdaf9e17c0
p8: add gtest cases for media encoder. r=jya

Comment 10

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/725677ef4b8e
https://hg.mozilla.org/mozilla-central/rev/d1255f131fe4
https://hg.mozilla.org/mozilla-central/rev/4dd46901846a
https://hg.mozilla.org/mozilla-central/rev/6dbd2767e7ab
https://hg.mozilla.org/mozilla-central/rev/3bf7f358af16
https://hg.mozilla.org/mozilla-central/rev/b86195e2307a
https://hg.mozilla.org/mozilla-central/rev/8759840bbdd3
https://hg.mozilla.org/mozilla-central/rev/74bdaf9e17c0