[webkit-reviews] review requested: [Bug 204276] Implement the 'ic' unit from CSS Values 4 : [Attachment 437944] Patch

[bugzilla-daemon at webkit.org]

Kiet Ho <tho22 at apple.com> has asked  for review:
Bug 204276: Implement the 'ic' unit from CSS Values 4
https://bugs.webkit.org/show_bug.cgi?id=204276

Attachment 437944: Patch

https://bugs.webkit.org/attachment.cgi?id=437944&action=review




--- Comment #14 from Kiet Ho <tho22 at apple.com> ---
Created attachment 437944

  --> https://bugs.webkit.org/attachment.cgi?id=437944&action=review

Patch

Notes:
1. I think I'll need more guidance on what to do regarding
DeprecatedCSSOMPrimitive Value
2. Font::platformGlyphInit() requires both the glyph number and width of the
space character, therefore even if there's a method to get the glyph number
then width, we'd have to duplicate the logic to get the glyph number
regardless. I decided to just clean up the method and surrounding areas.