This bug was filed from the Socorro interface and is
report bp-e36766b9-17d2-4593-89b6-ecaa12120519 .
We have to do state.mImageRenderer.GetContainer() early in nsDisplayBackground::TryOptimizeToImageLayer and if it returns null, don't optimize into a layer.
This crash only happens when gpu image scaling is enabled, which is OFF by default so product and trunk aren't affected.
Created attachment 625440 [details] [diff] [review]
Comment on attachment 625440 [details] [diff] [review]
I would pull this into a local and then use that further down, but up to you.
e/contenteditable.css ../../../layout/style/designmode.css ../../../layout/style/TopLevelImageDocument.css ../../../layout/style/TopLevelVideoDocument.css ../../dist/bin/res
/Users/gal/workspace/mozilla-central/layout/base/nsDisplayList.cpp: In member function ‘bool nsDisplayBackground::TryOptimizeToImageLayer(nsDisplayListBuilder*)’:
/Users/gal/workspace/mozilla-central/layout/base/nsDisplayList.cpp:1168: error: no match for ‘operator!’ in ‘!state.nsBackgroundLayerState::mImageRenderer. nsImageRenderer::GetContainer()’
/Users/gal/workspace/mozilla-central/layout/base/nsDisplayList.cpp:1168: note: candidates are: operator!(bool) <built-in>
Created attachment 625481 [details] [diff] [review]
This page often crashes for me without the patch:
It has a large scaled background images and it loads pretty slowly, causing us to paint before the image is ready, resulting in a crash.
Works with my version of the patch. Lets land it.
Yeah, I made the same change locally. I don't have L3 access so I can't actually land anything.
Created attachment 625509 [details] [diff] [review]
I am not of much use these days, but comment 6 I can fix!