Created attachment 195955 [details]
reproduced html content
You need to define ENABLE_CSS3_TEXT, so nightly build cannot reproduce this.
In the attached content, the underline of "big p" is far from its bottom.
Created attachment 195956 [details]
Could you review this patch?
This patch fix the bug which you reviewed at the following commit:
Created attachment 278018 [details]
Created attachment 278019 [details]
Created attachment 278020 [details]
As you can see from the screenshots I just uploaded, all the browsers vary both the position and thickness of underlines based on font size.
This patch (195956) does not apply any more.
Comment on attachment 195956 [details]
We need to make sure that the following content places the two underlines differently with different thicknesses:
<p style="text-decoration: underline; font-size: 90px;">Hello</p>
<p style="text-decoration: underline; font-size: 10px;">Hello</p>
The patch, as its posted, does not appear to do that (although I can't tell for sure because I can no longer apply it)