Closed Bug 1000736 Opened 7 years ago Closed 7 years ago

Throw a not supported error if an app tries to record a video from a video file being transcoded into a stream


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

30 Branch
Gonk (Firefox OS)
Not set



blocking-b2g 1.4+
Tracking Status
firefox30 --- wontfix
firefox31 --- wontfix
firefox32 --- fixed
b2g-v1.4 --- fixed
b2g-v2.0 --- fixed


(Reporter: jsmith, Assigned: rlin)



(2 files, 1 obsolete file)

1.4 wasn't intending to support video recording for transcoded video files, but it's currently exposed to the web. This is potentially bad for a few reasons:

1. Poor developer experience - a developer could think that the feature works, as some parts of video recording work here, but others do not.
2. Unintentional user side effects - a user gets negatively impacted from an app using an unsupported API with an unexpected side effect (e.g. crash)

As such, we should throw a not supported error if someone tries to record a video using a transcoded file to avoid getting into this situation in the first place.

Reference Bug List:

blocking-b2g: --- → 1.4?
Assignee: nobody → rlin
Hi Roc, 
Do you know how can I distinguish the media stream come from video tag or mozCamera?
blocking-b2g: 1.4? → 1.4+
ni roc.
Flags: needinfo?(roc)
What's actually causing the problems? Unexpected frame sizes or frame rates? If so we should simply check for the frame sizes and rates we want to support from the camera and reject everything else. I think we should do that instead of trying to detect transcoding specifically.
Flags: needinfo?(roc)
The encoder receive a new color format from omx decoder: OMX_QCOM_COLOR_FormatYUV420PackedSemiPlanar32m = 0x7FA30C04, the OMXWrapper need to handle this.
Attached patch patch v1 (obsolete) — Splinter Review
Check if the frame size is different with the gUM frame size and throw error.
Also add check for the color format.
Attachment #8412533 - Flags: review?(roc)
Attachment #8412533 - Flags: feedback?(cku)
Comment on attachment 8412533 [details] [diff] [review]
patch v1

Thanks derf, 
The perf should be readed in mainthread. Cancel review.
Attachment #8412533 - Flags: review?(roc)
Attachment #8412533 - Flags: feedback?(cku)
Attached patch bug.patchSplinter Review
Simple way to block the source form video tags by checking image format.
Attachment #8413657 - Flags: review?(tterribe)
Attachment #8412533 - Attachment is obsolete: true
Attachment #8413657 - Flags: review?(tterribe) → review+
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla32
You need to log in before you can comment on or make changes to this bug.