Gecko assertion failures when using Stagefright decoding: "ScaleYCbCrToRGB565 source image unpadded?"

RESOLVED INACTIVE

Status

()

RESOLVED INACTIVE
6 years ago
5 months ago

People

(Reporter: cpeterson, Unassigned)

Tracking

({assertion, qawanted})

Trunk
ARM
Android
assertion, qawanted
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [swdecoder])

(Reporter)

Description

6 years ago
When using Stagefright's hardware or software decoders in a debug build of Firefox, I get many assertion failures from our color conversion code:

I/Gecko   (16185): ###!!! ASSERTION: ScaleYCbCrToRGB565 source image unpadded?: 'abs(y_pitch) >= abs(source_width)+16', file /Users/cpeterson/Code/mozilla/central/gfx/ycbcr/ycbcr_to_rgb565.cpp, line 338
I/Gecko   (16185): ###!!! ASSERTION: ScaleYCbCrToRGB565 source image unpadded?: 'abs(y_pitch) >= abs(source_width)+16', file /Users/cpeterson/Code/mozilla/central/gfx/ycbcr/ycbcr_to_rgb565.cpp, line 338
(Reporter)

Updated

6 years ago
Depends on: 785339
(Reporter)

Comment 1

6 years ago
I believe these assertion failures are fallout from gralloc bug 767480.
Blocks: 767480, 785339
No longer depends on: 785339

Updated

6 years ago
Keywords: assertion
Whiteboard: [swdecoder]
It looks like this may be fixed by bug 785339.  Can we get this retested?  Thanks.
Keywords: qawanted
Kevin, the qawanted is for Maire's comment 2
Any specific device or video that this needs to be tested on? Besides an ICS or JB device.
(In reply to Kevin Brosnan [:kbrosnan] from comment #4)
> Any specific device or video that this needs to be tested on? Besides an ICS
> or JB device.

cpeterson -- what device did you see these assertions on?
(Reporter)

Comment 6

6 years ago
I think I repro'd the problem on my Galaxy S III, but not on my Galaxy Nexus.
Component: Audio/Video → Audio/Video: Playback
Mass closing do to inactivity.
Feel free to re-open if still needed.
Status: NEW → RESOLVED
Last Resolved: 5 months ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.