Sylvain Pasche Reporter
	Description • 14 years ago

Testcase: data:text/html,<i>"Lorem</i>

The "L" glyph is positioned at the same place as the double quote, as if the double quote had no width. I can see it on Windows 7, but not on XP or Linux.

Regression range: 
Works: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a3pre) Gecko/20100315 Minefield/3.7a3pre
Broken: Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.3a4pre) Gecko/20100316 Minefield/3.7a4pre

http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=17a3e86d5ec5&tochange=050887c64183

	Boris Zbarsky [:bzbarsky]
	Comment 1 • 14 years ago

Likely regression from bug 502906?

	Jonathan Kew [:jfkthame] Assignee
	Comment 2 • 14 years ago

I think this is a Windows 7 Uniscribe and/or Times New Roman font bug. The same problem occurs with Times New Roman Italic in Notepad on Win7.

Bug 502906 exposed this because it caused us to use the Uniscribe rendering path by default, but it's not really our bug.

I see the same problem with the directwrite font backend, incidentally, so even if we revert to using GDI rendering by default, the problem will persist/return as we move to dwrite.

The harfbuzz backend does not have this problem, so setting gfx.font_rendering.harfbuzz.level = 1 on a trunk build will resolve it.

	JeffEvarts
	Comment 4 • 14 years ago

See also:
436087
491389
540475
573407
This bug has been around long enough, and still happens on the latest & greatest version. Let's confirm and get started on a fix.

	Boris Zbarsky [:bzbarsky]
	Comment 5 • 14 years ago

JeffEvarts, the bug this bug report is about is a bug in Windows itself.  See comment 3.  The only way to "fix" is to somehow detect the OS-level braindamage and try to do "something else".  Whatever that would be...

	Joe Drew (not getting mail)
	Comment 6 • 14 years ago

Does this occur in other browsers?

	Benjamin Smedberg
	Comment 7 • 14 years ago

Doesn't occur in Chrome or IE8, but do they use uniscribe?

Could we just pick a different default serif font, such as Georgia? It sounds like we need to do *something* here, since "L isn't an especially uncommon combination.

	Jonathan Kew [:jfkthame] Assignee
	Comment 8 • 14 years ago

(In reply to comment #7)
> Could we just pick a different default serif font, such as Georgia? It sounds
> like we need to do *something* here, since "L isn't an especially uncommon
> combination.

We could, although that wouldn't help in the (common, I'd guess) case of pages where the CSS explicitly calls for Times.

I think changing our default font would need to be a UX decision, not purely an engineering one.

	Jonathan Kew [:jfkthame] Assignee
	Comment 9 • 14 years ago

The root of the problem seems to be an error in the timesi.ttf font shipped on Windows. In the GDEF (glyph definition) table, glyph ID 5 "quotedbl" is marked as being class 3, which is defined (in http://www.microsoft.com/typography/otspec/GDEF.htm) as being "Mark glyph (non-spacing combining glyph)". This causes Uniscribe to ignore the advance width of the glyph as found in the font's metrics table, and force it to be rendered as zero-width instead.

	Jonathan Kew [:jfkthame] Assignee
	Comment 13 • 14 years ago

This is fixed as a result of bug 569531: now that we're using harfbuzz on Windows, the problem no longer occurs.