[webkit-reviews] review denied: [Bug 24101] Chromium Windows port should support semitransparent ClearType fonts : [Attachment 27982] Patch v2

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 26 11:44:26 PST 2009


Eric Seidel <eric at webkit.org> has denied Brett Wilson (Google)
<brettw at chromium.org>'s request for review:
Bug 24101: Chromium Windows port should support semitransparent ClearType fonts
https://bugs.webkit.org/show_bug.cgi?id=24101

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

------- Additional Comments from Eric Seidel <eric at webkit.org>
Unless I'm reading this wrong, there is no savings in the m_cachedBuffers
logic.	I would suggest just removing it.  It seems you blast away the
m_cachedBuffers every time, and never re-use them.  So might as well just
always use m_ownedBuffers.


More information about the webkit-reviews mailing list