[Z3C] Green/Flash images when playing FTU tutorial video with OpenMax PDM enabled.

RESOLVED FIXED in Firefox 46

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jhlin, Assigned: jhlin)

Tracking

Trunk
mozilla46
Unspecified
Gonk (Firefox OS)
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox46 fixed)

Details

Attachments

(1 attachment, 1 obsolete attachment)

STR:
1. Goto 'Settings' > 'Developer'
2. Click 'Launch First Time Use'
3. Clicking 'Next' all the way to video tutorial page. The video will start playing, but it shows green image and flash.
Currently OmxDataDecoder always disable/reenable port when receiving OMX_EventPortSettingsChanged event [1].
On Z3C, decoder will first generate OMX_EventPortSettingsChanged(nData2: OMX_IndexParamPortDefinition), and after the port is reenabled, it will generate another OMX_EventPortSettingsChanged(nData2: OMX_IndexConfigCommonOutputCrop), which causes OmxDataDecoder to disable/reenable the port again.

Stagefright addresses this by examining nData2 and not always disabling/reenabling the port [2].

[1] https://dxr.mozilla.org/mozilla-central/source/dom/media/platforms/omx/OmxDataDecoder.cpp?from=OmxDataDecoder.cpp#659
[2] https://android.googlesource.com/platform/frameworks/av/+/kitkat-release/media/libstagefright/ACodec.cpp#4241
Attachment #8702828 - Flags: feedback?(ayang)
Attachment #8702828 - Flags: feedback?(ayang) → feedback+
Attachment #8702828 - Flags: review?(sotaro.ikeda.g)
Comment on attachment 8702828 [details] [diff] [review]
Handle OMX_EventPortSettingsChanged like ACodec.

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

Looks good :)
Attachment #8702828 - Flags: review?(sotaro.ikeda.g) → review+
Carry r+ from sotaro.
Attachment #8702828 - Attachment is obsolete: true
Attachment #8707715 - Flags: review+
https://hg.mozilla.org/mozilla-central/rev/ab3c6ba63c89
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.