946215 - [MediaEncoder] OMXCodecWrapper should receive and handle error message from MediaCodec

Status: NEW

(Core :: Audio/Video: Recording, defect, P3)

Description

[u459114]

With HW OMX codec, we should not assume MediaCodec works correctly all the time. OMXCodecWrapper should receive, error message sent by MeidaCodec, and notify MediaEncoder/ MediaRecoder to handle unexpected error.

Comment 1

[Randy Lin [:rlin] - Be Mozillian on 2015/01/26]

If media stream video frame size is larger than codec acceptance. 
We should stop the recorder. 
Test on emulator-jb. 
1. Get Fake gum with video size = 640*480
2. recording that media stream 
3. got fail and can't auto stop encoder
I/ACodec  (  404): setupVideoEncoder succeeded
I/OMXClient(  404): Using client-side OMX mux.
E/SoftAVCEncoder(  404): Failed to initialize the encoder: -8
E/ACodec  (  404): [OMX.google.h264.encoder] ERROR(0x80001001)
E/MediaCodec(  404): Codec reported an error. (omx error 0x80001001, internalError -2147483648)
E/OMXCodecWrapper(  404): MediaCodec error:-38
E/OMXCodecWrapper(  404): MediaCodec error:-38
E/OMXCodecWrapper(  404): MediaCodec error:-38
E/OMXCodecWrapper(  404): MediaCodec error:-38
E/OMXCodecWrapper(  404): MediaCodec error:-38
E/OMXCodecWrapper(  404): MediaCodec error:-38
E/OMXCodecWrapper(  404): MediaCodec error:-38

recorder status is still in recording.

Comment 2

[Bulk Bug Changes for mreavy's org]

Mass change P2->P3 to align with new Mozilla triage process.