[webkit-reviews] review granted: [Bug 15177] [gtk] FontPlatformData contains more fields than required : [Attachment 16254] Final Patch (all in one)

[bugzilla-daemon at webkit.org]

Mark Rowe (bdash) <mrowe at apple.com> has granted Sven Herzberg
<sven at imendio.com>'s request for review:
Bug 15177: [gtk] FontPlatformData contains more fields than required
http://bugs.webkit.org/show_bug.cgi?id=15177

Attachment 16254: Final Patch (all in one)
http://bugs.webkit.org/attachment.cgi?id=16254&action=edit

------- Additional Comments from Mark Rowe (bdash) <mrowe at apple.com>
Some minor comments:
The local variables inside FontPlatformData::FontPlatformData should lose the
m_ prefix, which is used only for members.  I think the declarations can be
moved to their point of first use too.
Our coding style also requires no space between the function name and it's
arguments.

I'll make those minor tweaks when I land this.