Closed Bug 388342 Opened 15 years ago Closed 15 years ago

imglib warning when loading jpeg images - WARNING: NS_ENSURE_TRUE(index < static_cast<PRUint32>(mFrames.Count()))

Categories

(Core :: ImageLib, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: tor, Unassigned)

Details

Attachments

(1 file)

The patch that added APNG support added a NS_ENSURE_ARG to imgContainer::GetFrameAt().  Unfortunately the JPEG decoder triggers that because it always tries to get frame 0 (for mixed-multipart?), so using a debug build for dogfooding can get chatty at times with all the warnings that line emits.  This patch allows requesting in a container that doesn't yet contain a frame, and makes sure GetFrameAt always sets the return properly.
Attachment #272541 - Flags: review?(pavlov)
Attachment #272541 - Flags: review?(pavlov) → review+
Checked in.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.