Bug 42342 – Font download error for an @font-face rule invalidates other @font-face rules for the same font-family

• Component: CSS
• Status: RESOLVED FIXED

[Yuzo Fujishima]

Assume that we have the following file and CannotBeDownloaded.ttf cannot be downloaded
while Ahem.ttf can.

As per http://lists.w3.org/Archives/Public/www-style/2010Jul/0253.html , the first rule
must remain in effect while the second rule must be ignored.

WebKit r63408 ignores both rules.

Note that if
  src: url(CannotBeDownloaded.ttf);
is replaced with
  src: local(NonExistentLocalFont);
or
  src: url(CannotBeDownloaded.ttf), local(Arial);
, the first rule remains in effect and WebKit renders the file properly.


<style>
@font-face {
  font-family:myfont;
  src: url(Ahem.ttf);
}
@font-face {
  font-family:myfont;
  src: url(CannotBeDownloaded.ttf);
  unicode-range: U+30-39;
}
</style>
<p>
The first @font-face rule above should remain in effect
even if the font for the second @font-face rule cannot be downloaded.
Accordingly, the following text should be shown in Ahem.
</p>
<span style="font-family:myfont">
This must be shown in Ahem.
</span>

[Yuzo Fujishima]

Created attachment 61641 [details]
Proposed fix

Comment on attachment 61641 [details]
Proposed fix

Please fill in the WebCore change log.

[Yuzo Fujishima]

Created attachment 62032 [details]
Patch

[Yuzo Fujishima]

Hi,

Thank you for the review.
In WebCore/ChangeLog, I've added a comment to each method.

Yuzo

[Yuzo Fujishima]

Committed r63719: <http://trac.webkit.org/changeset/63719>

[Csaba Osztrogonác]

(In reply to comment #5)
> Committed r63719: <http://trac.webkit.org/changeset/63719>

fast/css/font-face-download-error.html 
introduced in r63719 fails on Qt bot:
http://build.webkit.org/results/Qt%20Linux%20Release/r63720%20%2815746%29/fast/css/font-face-download-error-pretty-diff.html

Could you check it, please?

[Yuzo Fujishima]

Committed r63722: <http://trac.webkit.org/changeset/63722>