[Webkit-unassigned] [Bug 52311] [chromium] Add command-line flag to enable composite to offscreen texture.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=52311





--- Comment #14 from Vangelis Kokkevis <vangelis at chromium.org>  2011-01-13 10:27:44 PST ---
(From update of attachment 78809)
View in context: https://bugs.webkit.org/attachment.cgi?id=78809&action=review

> Source/WebCore/platform/graphics/chromium/LayerRendererChromium.cpp:639
> +        setDrawViewportRect(IntRect(0, 0, m_rootLayerTextureWidth, m_rootLayerTextureHeight), true);

Could the two calls above be replaced by "useRenderSurface(0)" ?

> Source/WebCore/platform/graphics/chromium/LayerRendererChromium.cpp:643
> +        m_context->clearColor(1, 0, 0, 1); // Clear to red to detect regions not composite.

"not composite" -> "not composited" ?

> Source/WebCore/platform/graphics/chromium/LayerRendererChromium.cpp:658
> +        m_currentRenderSurface = 0; // Need to do this, or else next call to useRenderSurface will still be bound to

I don't believe you need to make this call now.  It should be taken care of when calling useRenderSurface(0)

> WebKit/chromium/src/WebViewImpl.cpp:972
> +        m_layerRenderer->setCompositeOffscreen(settings()->compositeToTextureEnabled()); // Clears display.

You probably need to define a separate clearDisplay() method in m_layerRenderer as setCompositeOffScreen uses m_rootLayerTextureWidth/Height which won't be correct while resizing.  However, I'm getting a bit confused now as to why we need to clear the window.. DrawLayers does a clear that affects the alpha channel so the resulting offscreen texture should always have alpha = 1.  Why do we need to clear again?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.