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>
Created attachment 61641 [details] Proposed fix
Comment on attachment 61641 [details] Proposed fix Please fill in the WebCore change log.
Created attachment 62032 [details] Patch
Hi, Thank you for the review. In WebCore/ChangeLog, I've added a comment to each method. Yuzo
Committed r63719: <http://trac.webkit.org/changeset/63719>
(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?
Committed r63722: <http://trac.webkit.org/changeset/63722>