[webkit-reviews] review granted: [Bug 84059] Re-implement backFaceVisibility to avoid dealing with perspective w < 0 problem : [Attachment 138393] Patch

[bugzilla-daemon at webkit.org]

Adrienne Walker <enne at google.com> has granted Shawn Singh
<shawnsingh at chromium.org>'s request for review:
Bug 84059: Re-implement backFaceVisibility to avoid dealing with perspective w
< 0 problem
https://bugs.webkit.org/show_bug.cgi?id=84059

Attachment 138393: Patch
https://bugs.webkit.org/attachment.cgi?id=138393&action=review

------- Additional Comments from Adrienne Walker <enne at google.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=138393&action=review


Ah, I see.  The inverse does make all the difference.  Thanks for the
explanation.

> Source/WebCore/platform/graphics/transforms/TransformationMatrix.h:364
> +    bool isBackFaceVisible() const;
> +

This definitely needs a comment.  The back face of what?  With respect to what?