keeleysam / tenfourfox

Automatically exported from code.google.com/p/tenfourfox
0 stars 0 forks source link

Font sanitizer has pathologic fontogermophobia #146

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The test case in 
https://bug457194.bugzilla.mozilla.org/attachment.cgi?id=340544 shows an X and 
it should show a blank square. Regressed from 11. Stable is not affected.

Original issue reported on code.google.com by classi...@floodgap.com on 21 Apr 2012 at 5:24

GoogleCodeExporter commented 9 years ago
Actually, this seems to affect all web fonts. wiki.mozilla.org is also in the 
wrong font.

Original comment by classi...@floodgap.com on 22 Apr 2012 at 3:51

GoogleCodeExporter commented 9 years ago
It's the sanitizer.

Timestamp: 4/21/12 9:04:33 PM
Error: downloadable font: rejected by sanitizer (font-family: "AcidAhemTest" 
style:normal weight:normal stretch:normal src index:0)
source: http://acid3.acidtests.org/font.ttf
Source Code:
@font-face {   font-family: "AcidAhemTest";   src: url("font.ttf"); }

Original comment by classi...@floodgap.com on 22 Apr 2012 at 4:06

GoogleCodeExporter commented 9 years ago
M712217 could cause this and is not in ESR. M739925 is in ESR, so it can't be 
that.

Original comment by classi...@floodgap.com on 22 Apr 2012 at 4:09

GoogleCodeExporter commented 9 years ago
+// Use a macro instead of a function because gcc 4.4.3 creates static
+// initializers in that case. Note this macro assumes a little-endian system.
+#define TAG(a, b, c, d) (a | (b << 8) | (c << 16) | (d << 24))

Well, there we go.

Original comment by classi...@floodgap.com on 22 Apr 2012 at 4:12

GoogleCodeExporter commented 9 years ago
upstreamed as https://bugzilla.mozilla.org/show_bug.cgi?id=747816

Original comment by classi...@floodgap.com on 23 Apr 2012 at 3:29

GoogleCodeExporter commented 9 years ago
and http://code.google.com/p/chromium/issues/detail?id=124629

Original comment by classi...@floodgap.com on 23 Apr 2012 at 1:40

GoogleCodeExporter commented 9 years ago

Original comment by classi...@floodgap.com on 23 Apr 2012 at 1:40

GoogleCodeExporter commented 9 years ago
Temporary fix in for 13. Final fix will be taken in the next upstream swap.

Original comment by classi...@floodgap.com on 19 May 2012 at 4:57

GoogleCodeExporter commented 9 years ago

Original comment by classi...@floodgap.com on 5 Jun 2012 at 8:35

GoogleCodeExporter commented 9 years ago
The final fix is in Fx15, and works, so this is closed.

Original comment by classi...@floodgap.com on 18 Jul 2012 at 7:40