[Webkit-unassigned] [Bug 169576] Follow-up (r213833): write a layout test for 169199

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Tue Mar 14 10:31:11 PDT 2017


--- Comment #2 from Miguel Gomez <magomez at igalia.com> ---
I'm working on this.

I have the test that deletes the decoder and works fine. But without r213833 it doesn't necessarily cause a crash, as it's a race condition.

With the implementation of BitmapImage::clearDecoder() you suggest, ImageSource::clear() calls setData() just after setting the ImageFrameCache decoder to nullptr, and that setData() creates a new decoder that the decoding thread will use without crashing.

The crash happens only when the decoding thread is using the decoder exactly between its destruction and the creation of the new one, which is pretty hard to reproduce. I could force it though, using an implementation of BitmapImage::clearDecoder() that just deletes the decoder and doesn't cause setData() to be called.

Should I do that? Or should I use the implementation you suggest, even if the crash is not easily reproducible without r213833?

You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20170314/4e884b24/attachment.html>

More information about the webkit-unassigned mailing list