[webkit-reviews] review requested: [Bug 204276] Implement the 'ic' unit from CSS Values 4 : [Attachment 437950] 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 437950: Patch

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




--- Comment #15 from Kiet Ho <tho22 at apple.com> ---
Created attachment 437950

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

Patch

Notes:\n1. I think I'll need more guidance on what to do regarding
DeprecatedCSSOMPrimitive Value\n2. 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.