[webkit-reviews] review denied: [Bug 4355] use small-caps variant font if it's available : [Attachment 3287] small-caps patch 1.0

[bugzilla-request-daemon at opendarwin.org]

Darin Adler <darin at apple.com> has denied Nicholas Shanks
<contact at nickshanks.com>'s request for review:
Bug 4355: use small-caps variant font if it's available
http://bugzilla.opendarwin.org/show_bug.cgi?id=4355

Attachment 3287: small-caps patch 1.0
http://bugzilla.opendarwin.org/attachment.cgi?id=3287&action=edit

------- Additional Comments from Darin Adler <darin at apple.com>
Please fix formatting. You need a space between the "if" and the "(" on lines
like:

+		 if([renderer _capitalizeSmallCaps])

Also, seems better to just make the capitalizeSmallCaps field @public rather
than calling a method on every character to check that boolean.

Otherwise, looks good, and we can do review+ if we also have a layout test
demonstrating the correct behavior -- they metrics in the layout test will be
the telltale about whether the correct font and glyph was chosen.