[Webkit-unassigned] [Bug 70634] Mark GraphicsLayers as opaque when possible

[bugzilla-daemon at webkit.org]

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





--- Comment #25 from Dana Jansens <danakj at chromium.org>  2011-10-27 16:54:13 PST ---
Regarding testing:

I think the right way to test this is to add contentsOpaque to the output of GraphicsLayer::dumpProperties().  However this change alone which break and require rebaselining a *lot* of layout tests.

There are many ways this could go and I'd like to know what you all think is the best.

First I'll say my own thoughts:  Currently LayoutTestController.layoutTreeAsText(bool) takes an argument to show debug info.  If we extend this to allow setting various flags for layoutTreeAsText(), we can enable "contentsOpaque" in the DRT output only when we want it, and expose this to tests.

To do this:
- Remove boolean argument to layoutTreeAsText().
- Add setLayerTreeAsTextFlag(string, bool) to LayoutTestController.
  - Eventually this string is parsed to a value from the enum LayerTreeAsTextBehaviourFlags in GraphicsLayer.h

The big win here is that we can use DRT output to test this property nicely, without having to rebaseline any other tests.  If/when future properties are flipped on in GraphicsLayer or equivalent, this could pay off there also.

I will attach a patch to give an idea what this would look like.

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