[Webkit-unassigned] [Bug 88908] [chromium] webkit-backface-visibility doesn't work with <video>

[bugzilla-daemon at webkit.org]

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





--- Comment #20 from Shawn Singh <shawnsingh at chromium.org>  2012-06-14 15:55:21 PST ---
(From update of attachment 147668)
Few comments in addition to the testing todo =)


View in context: https://bugs.webkit.org/attachment.cgi?id=147668&action=review

> Source/Platform/chromium/public/WebLayer.h:112
> +    // Set whether this layer is a content layer (e.g, canvas, plugin, WebGL, or video)

probably should update this comment

> Source/WebCore/platform/graphics/chromium/cc/CCLayerTreeHostCommon.cpp:250
> +        layer = layer->parent();

I feel uncomfortable doing this, what if someone blindly adds another condition for skpping the layer below this poitn in code?   Perhaps its better to make the two cases more explicit without changing any state (i.e. which layer is being considered) within this function.

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