1038615 - Report GMP decoding/encoding errors asynchronously

Status: RESOLVED FIXED

(Core :: Audio/Video, defect)

Description

[Chris Pearce [:cpearce (Not reading bugmail)]]

The functions on the GMPVideoDecoder interface have return values, but the return values are ignored by the plugin harness, and not returned to Gecko. The return valued can't be sent back to Gecko because the methods are defined to be async in the IPDL definitions. There also isn't any other way to report errors from the decoder or encoder to Gecko. So we should have a callback to report errors, and the return values of the functions on GMPVideoDecoder and friends should be void.

Comment 1

[Chris Pearce [:cpearce (Not reading bugmail)]]

Attachment: Patch

Comment 2

[Ethan Hugg [:ehugg]]

Comment on attachment 8456038 [details] [diff] [review]
Patch

Review of attachment 8456038 [details] [diff] [review]:
-----------------------------------------------------------------

This should be fine.  I need some warning as to when this will land though it will break our compat and build system so I need to send out some notice and have our patches ready.   I assume you're targeting this for FF33.

Comment 3

[Ethan Hugg [:ehugg]]

Attachment: Patch to OpenH264 to match these changes

Comment 4

[Ethan Hugg [:ehugg]]

Comment on attachment 8457439 [details] [diff] [review]
Patch to OpenH264 to match these changes

Review of attachment 8457439 [details] [diff] [review]:
-----------------------------------------------------------------

I will be proposing these changes into RBCommons when the FF patch hits M-C.  You can apply this diff onto openh264 master or if it's easier just pull the github/ethanhugg/gmpapi-errorchanges branch.  Adding jesup as feedback, but the formal review will be in rbcommons for OpenH264.

Comment 5

[Chris Pearce [:cpearce (Not reading bugmail)]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/fd72e28c6d38

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/fd72e28c6d38