[webkit-reviews] review requested: [Bug 16362] Incorrect implementation of 'ex' unit : [Attachment 21006] patch 2

[bugzilla-daemon at webkit.org]

Anatoli Papirovski <apapirovski at mac.com> has asked  for review:
Bug 16362: Incorrect implementation of	'ex' unit
http://bugs.webkit.org/show_bug.cgi?id=16362

Attachment 21006: patch 2
http://bugs.webkit.org/attachment.cgi?id=21006&action=edit

------- Additional Comments from Anatoli Papirovski <apapirovski at mac.com>
Ok, as discussed I've corrected the xheight calculation not to make ex = em
which was done to fix a gmail bug that no longer exists. I've also removed the
"ex" from margin declaration on input radio and checkbox.

I didn't include any test cases, because a lot (and I mean A LOT) of test cases
need to be regenerated, pretty much anything that uses "ex", "x-height", or
radio or checkbox form elements.

Also, those Microsoft test cases are actually wrong... per the CSS spec, if the
font does not have an xHeight specified, the rendering engine should try and
compute it from an actual x, thus the xHeight for Ahem is 0.8 rather than 0.5.