[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 #8 from James Robinson <jamesr at chromium.org>  2012-06-12 18:41:47 PST ---
(In reply to comment #7)
> (In reply to comment #6)
> > (In reply to comment #5)
> 
> > > let's add a layout test that reproduces the problem and gets fixed by this patch.
> > 
> > While you're doing that, could you construct test cases for other content layer types (canvas 2d, webgl, etc) and see if they suffer from the same issue?
> 
> Working on that now.  Btw, this does indeed reproduce with WebGL (and I assume it will reproduce with canvas2D)

Sounds quite likely - that implies doing a general fix for GraphicsLayerChromium::m_contentsLayer is likely to be what we want.

> > > > Source/Platform/chromium/public/WebContentLayer.h:-58
> > > > -    WEBKIT_EXPORT void setDoubleSided(bool);
> > > 
> > > I'm guessing we shouldn't remove this.
> > 
> > Terminology inversion here.
> > 
> > In the WebLayer or LayerChromium hierarchy, we use a different terminology.  *ContentLayer* means a layer type that draws via a paintContents() call.  
> 
> I'll need to understand these structures and their target forms a bit better.  In its current form, WebContentLayer is hard to distinguish from WebLayer.  I'll discuss more offline.

That's putting it mildly :).  Feel free to grab me and try to brainstorm an improvement.

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