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] Proposed patch
Comment on attachment 66470 [details] Proposed patch View in context: https://bugs.webkit.org/attachment.cgi?id=66470&action=prettypatch > WebCore/platform/graphics/chromium/LayerRendererChromium.cpp:83 > + 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>. > WebKit/chromium/src/WebViewImpl.cpp:2110 > + if (m_compositorCreationFailed) nit: I think the body of this function should just be changed to: return !m_compositorCreationFailed; R=me with those nits fixed
Committed r66777: <http://trac.webkit.org/changeset/66777>