If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Write C++ unit test to make sure we don't allow re-entrant sync decoding

RESOLVED INVALID

Status

()

Core
ImageLib
RESOLVED INVALID
7 years ago
3 years ago

People

(Reporter: bholley, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Followup bug from bug 580669, since we realized that everything we needed to test was noscript.
Created attachment 460321 [details] [diff] [review]
WIP tests

Here are the beginnings of a js test that I wrote before realizing that stuff was noscript.
Do you still plan on writing this test Bobby?
(In reply to Benoit Girard (:BenWa) from comment #2)
> Do you still plan on writing this test Bobby?

Nope! It's a valid bug (I think), but it seems unlikely that anyone's ever going to do it. I'll let joe make the call.
It's not wontfix, though, as I would totally take a patch. It's something someone will need a fair bit of knowledge about imagelib to implement.
This isn't even possible anymore. Sync decodes (much like windmills) do not work that way.
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.