Closed
Bug 786103
Opened 12 years ago
Closed 12 years ago
Stagefright: Galaxy S III hardware decoder video is green (OMX_COLOR_FormatYUV420SemiPlanar)
Categories
(Core :: Audio/Video, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla18
Tracking | Status | |
---|---|---|
firefox16 | --- | unaffected |
firefox17 | --- | fixed |
firefox18 | --- | fixed |
People
(Reporter: cpeterson, Assigned: cpeterson)
References
()
Details
(Whiteboard: [hwdecoder])
Attachments
(2 files)
90.68 KB,
image/jpeg
|
Details | |
1.30 KB,
patch
|
roc
:
review+
kanru
:
feedback+
bajaj
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
This is a color conversion regression in Nightly build 2012-08-27. The pushlog from build 08-26 to 08-27 is: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=b3cce81fef1a&tochange=8af6a22827ec This bug is likely a regression from this changeset for bug 767480: https://hg.mozilla.org/mozilla-central/rev/f26d6c5a2d93
Updated•12 years ago
|
Whiteboard: [swdecoder]
Updated•12 years ago
|
Whiteboard: [swdecoder] → [hwdecoder]
Comment 1•12 years ago
|
||
Same issue on the Galaxy Note 1 with hardware playback. Video is green with flickering in and out of non-green.
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → cpeterson
Status: NEW → ASSIGNED
Summary: Stagefright hardware decoded videos are green on Galaxy S III (OMX_COLOR_FormatYUV420SemiPlanar) → Stagefright: Galaxy S III hardware decoder video is green (OMX_COLOR_FormatYUV420SemiPlanar)
Assignee | ||
Comment 2•12 years ago
|
||
This bug is a regression from bug 767480: https://hg.mozilla.org/mozilla-central/rev/f26d6c5a2d93 That fix inadvertently moved some pointer math out of CopyPlane()'s loops. Because the aSrc and aDst pointers where not advanced, the Cb and Cr planes were not copied for color formats that hit CopyPlane()'s slow path. This patch fixes the green videos, but does not address related bug 785275.
Attachment #659068 -
Flags: review?(roc)
Attachment #659068 -
Flags: feedback?(kchen)
Comment on attachment 659068 [details] [diff] [review] advance-aSrc-aDst-pointers.patch Review of attachment 659068 [details] [diff] [review]: ----------------------------------------------------------------- oops
Attachment #659068 -
Flags: review?(roc) → review+
Updated•12 years ago
|
Attachment #659068 -
Flags: feedback?(kchen) → feedback+
Assignee | ||
Comment 4•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/f0f098ac4386
Target Milestone: mozilla17 → mozilla18
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f0f098ac4386
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 659068 [details] [diff] [review] advance-aSrc-aDst-pointers.patch [Approval Request Comment] Bug caused by (feature/regressing bug #): bug 767480 (which regressed Nightly 18 and Aurora 17) User impact if declined: H264 videos may be green on some Android devices Testing completed (on m-c, etc.): m-c Risk to taking this patch (and alternatives if risky): Low risk. This patch just reverts a copy/paste typo in a video color conversion function. String or UUID changes made by this patch: N/A
Attachment #659068 -
Flags: approval-mozilla-aurora?
Updated•12 years ago
|
Attachment #659068 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 7•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/8a23501c1cf0
You need to log in
before you can comment on or make changes to this bug.
Description
•