Currently if the accelerated compositor fails to initialize (e.g. failed to get a gl context), page renders without the composited layers. The correct behavior would be to switch over to software compositing.
Created attachment 66470 [details]
Comment on attachment 66470 [details]
View in context: https://bugs.webkit.org/attachment.cgi?id=66470&action=prettypatch
> + PassOwnPtr<LayerRendererChromium> layerRenderer(new LayerRendererChromium(gles2Context));
nit: this should be changed to use OwnPtr, like so:
OwnPtr<LayerRenderChromium> layerRenderer(new LayerRendererChromium(gles2Context));
Then you should use the .release() method of OwnPtr<T> to finally return a PassOwnPtr<T>.
> + if (m_compositorCreationFailed)
nit: I think the body of this function should just be changed to:
R=me with those nits fixed
Committed r66777: <http://trac.webkit.org/changeset/66777>