[Webkit-unassigned] [Bug 86546] [chromium] Back face culling doesn't handle perspective projection

[bugzilla-daemon at webkit.org]

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





--- Comment #1 from Shawn Singh <shawnsingh at chromium.org>  2012-05-15 17:02:34 PST ---
Hi Eric, is there a website or test case that fails?  some way to repro the problem?

I wonder if you're referring to m19 branch?   In the current code (just before m20 branch I think) there is no static inline bool backFaceIsVisible.  That was recently replaced with a new implementation in WebCore::TransformationMatrix, with unit tests in CCMathUtilTest.cpp.

We do have a unit test which I think already covers this case in CCMathUtilTest.cpp - verifyBackfaceVisibilityForPerspective, there's a nifty ASCII diagram that depicts what you might be describing and that test works correctly as far as I'm aware.

Does this address what you're saying?

Thanks!

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