104954 2012-12-13 14:31:00 -0800 http/tests/canvas/canvas-slow-font-loading.html fails on Mac 2015-02-24 03:57:50 -0800 1 1 1 Unclassified WebKit Canvas 528+ (Nightly build) Unspecified Unspecified NEW P2 Normal --- 1 bdakin webkit-unassigned bdakin cdumez kenneth thorton oldest_to_newest 790936 0 bdakin 2012-12-13 14:31:00 -0800 http/tests/canvas/canvas-slow-font-loading.html is failing on the Mac bots, and probably has been failing since it was added with http://trac.webkit.org/changeset/135888 It seems to pass in-browser. 790951 1 bdakin 2012-12-13 14:41:51 -0800 Skipped for now with http://trac.webkit.org/changeset/137656 1071613 2 247219 ddkilzer 2015-02-24 03:57:50 -0800 Created attachment 247219 Fixes the test, but ... This patch fixes the test, but I'm not sure if it's correct. - I think the test is failing because there is a built-in "Ahem" font that's loaded in DRT/WKTR, and I assume that built-in font will be preferred over a WebFont with the same name, so it always loads fast. If that is correct, the test shouldn't be using "Ahem" for the name of the WebFont. - If I change the name of the WebFont to "AhemSlow", the test passes, but I'm not sure if that's because it's doing the right thing, or whether the AhemSlow font isn't loading because the internal name of Ahem.ttf doesn't match "AhemSlow". Commenting out the "sleep(1)" command in the CGI script doesn't seem to change the results of the test, but maybe that's just an extra precaution. (In other words, can you give an arbitrary name to any *.ttf file loaded as a WebFont, regardless of its PostScript name?) 247219 2015-02-24 03:57:50 -0800 2015-02-24 03:57:50 -0800 Fixes the test, but ... bug-104954-possible-fix.diff text/plain 845 ddkilzer ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2h0dHAvdGVzdHMvY2FudmFzL2NhbnZhcy1zbG93LWZv bnQtbG9hZGluZy5odG1sIGIvTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9jYW52YXMvY2FudmFzLXNs b3ctZm9udC1sb2FkaW5nLmh0bWwKaW5kZXggYzUyNmMyOC4uM2NlNzE0NiAxMDA2NDQKLS0tIGEv TGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9jYW52YXMvY2FudmFzLXNsb3ctZm9udC1sb2FkaW5nLmh0 bWwKKysrIGIvTGF5b3V0VGVzdHMvaHR0cC90ZXN0cy9jYW52YXMvY2FudmFzLXNsb3ctZm9udC1s b2FkaW5nLmh0bWwKQEAgLTMsNyArMyw3IEBACiA8dGl0bGU+Q2FudmFzIHRlc3Q6IHNsb3cgZm9u dCBsb2FkaW5nPC90aXRsZT4KIDxzdHlsZT4KIEBmb250LWZhY2UgewotICBmb250LWZhbWlseTog QWhlbTsKKyAgZm9udC1mYW1pbHk6IEFoZW1TbG93OwogICBzcmM6IHVybCgiLi4vd2ViZm9udC9z bG93LWFoZW0tbG9hZGluZy5jZ2kiKTsKIH0KIDwvc3R5bGU+CkBAIC0yMCw3ICsyMCw3IEBAIHZh ciBjdHggPSBjYW52YXMuZ2V0Q29udGV4dCgnMmQnKTsKIC8vIGluIHRpbWUgZm9yIGRyYXdpbmcg dGhlIHRleHQuIEFzIGEgY29uc2VxdWVuY2UsIHRoZQogLy8gZmFsbGJhY2sgZm9udCBzaG91bGQg YmUgdXNlZCBhbmQgdGhlIHJlc3VsdCBzaG91bGQKIC8vIGJlIDIgZ3JlZW4gJ1gnIG9uIGEgcmVk IGJhY2tncm91bmQuCi1jdHguZm9udCA9ICc2N3B4IEFoZW0sIEFyaWFsJzsKK2N0eC5mb250ID0g JzY3cHggQWhlbVNsb3csIEFyaWFsJzsKIGN0eC5maWxsU3R5bGUgPSAnI2YwMCc7CiBjdHguZmls bFJlY3QoMCwgMCwgMTAwLCA1MCk7CiBjdHguZmlsbFN0eWxlID0gJyMwZjAnOwo=