Open Bug 946215 Opened 11 years ago Updated 2 years ago

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

Categories

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

ARM
Gonk (Firefox OS)
defect

Tracking

()

People

(Reporter: u459114, Assigned: jhlin)

References

(Blocks 1 open bug)

Details

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.
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.
Component: Video/Audio → Video/Audio: Recording
Rank: 25
Priority: -- → P2
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.