REGRESSION (r107516): fast/css/font-face-default-font.html is failing

• Component: Layout and Rendering
• Status: RESOLVED FIXED

<http://trac.webkit.org/r107516> caused fast/css/font-face-default.font.html to fail. Specifically, the italicized text changed from Arial to Times.

[Kenichi Ishibashi]

According to the latest draft spec:
- the default value of font-style descriptor is normal (http://dev.w3.org/csswg/css3-fonts/#font-prop-desc)
- a font which has desirable style (italic in this case) is prioritized others if the font-family is matched (http://dev.w3.org/csswg/css3-fonts/#font-style-matching)

fast/css/font-face-default-font.html defines @font-face without font-style:

@font-face {
    font-family: Times;
    src: local("Arial");
}

And Mac (and maybe other platforms) has regular and italic Times font. These facts caused the test failure.

I think the current behavior follows the draft spec. Just adding following @font-face will solve the failure.

@font-face {
    font-family: Times;
    font-style: italic;
    src: local("Arial");
}

Makes sense.

[Kenichi Ishibashi]

Created attachment 128373 [details]
Patch

Comment on attachment 128373 [details]
Patch

Not really necessary for the test, but easier than updating all the results.

[Kenichi Ishibashi]

Comment on attachment 128373 [details]
Patch

Thank you!

[WebKit Review Bot]

Comment on attachment 128373 [details]
Patch

Clearing flags on attachment: 128373

Committed r108609: <http://trac.webkit.org/changeset/108609>

[WebKit Review Bot]

All reviewed patches have been landed.  Closing bug.