STEPS TO REPRODUCE: Load the testcase. If the image shows up, shift-reload This is also what bites chatzilla's smilies. The problem is that we end up with a state change from LOADING to not LOADING for the generated content, which tries to reframe it... which doesn't really work because it's not real content, so it can't be usefully removed from parent or added to the parent. The two options I see are either reframing the parent or not reframing the generated content here. The latter is simpler for now; the former would be needed to do generated content fallback.

We need both parts, since nsImageLoadingContent just notifies on the ImageState() difference, not the IntrinsicState() difference. Perhaps we should change that...

Comment on attachment 200660 [details] [diff] [review] Patch I guess. I'd prefer it if reframing was fixed to correctly handle this... but if that's too hard, please at least add a comment to these two places explaining why you do this. r=biesi

The real problem is lack of clear idea of what "correctly" is here. I'd rather not change layout in ways that depend on imagelib being sometimes sync, which is what I'd need here if I reframed the parent on LOADING state change.

Fixed.