Bug 24220

Summary:

GCGetGlyphIdealMetrics fails and causes random text

Product:

WebKit

Reporter:

Eli Loewen <flclfan>

Component:

New Bugs

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED WORKSFORME

Severity:

Normal

CC:

erik-webkit, hyatt, mitz, mrowe, zwarich

Priority:

Keywords:

InRadar, NeedsReduction

Version:

528+ (Nightly build)

Hardware:

Mac

OS:

OS X 10.5

URL:

http://www.eliloewen.com

Attachments:

Description	Flags
Console output of GCGetGlyphIdealMetrics errors	none
Reduction	none

Eli Loewen

Reported 2009-02-26 21:45:27 PST

I just got the nightly of webkit r41242 and tested it in my website and when I did all the text on the site was replaced by random text and characters with a ton of errors in the logs saying "2/26/09 9:38:08 PM Safari[728] GCGetGlyphIdealMetrics failed: error 1." Here is a pic of the page is you want: http://img9.imageshack.us/img9/1084/picture1glb.png

Attachments
Console output of GCGetGlyphIdealMetrics errors (26.55 KB, text/plain) 2009-02-27 08:48 PST, Cameron Zwarich (cpst)	no flags	Details
Reduction (192 bytes, text/html) 2009-02-27 08:58 PST, Cameron Zwarich (cpst)	no flags	Details
View All Add attachment proposed patch, testcase, etc.

Mark Rowe (bdash)

Comment 1 2009-02-26 22:12:32 PST

Wow, that's a nasty one!

Mark Rowe (bdash)

Comment 2 2009-02-26 22:12:44 PST

<rdar://problem/6629728>

mitz

Comment 3 2009-02-26 22:31:02 PST

The problem appears to be specific to the version of Geneva used on the webpage.

Mark Rowe (bdash)

Comment 4 2009-02-26 22:31:38 PST

I'm not sure that this is a regression any more as I realised that I wasn't testing what I thought I was testing.

Mark Rowe (bdash)

Comment 5 2009-02-26 22:33:08 PST

It's probably a violation of the license of the font to be redistributing it like this. The font even says © Apple.

Eli Loewen

Comment 6 2009-02-27 08:08:09 PST

(In reply to comment #5) > It's probably a violation of the license of the font to be redistributing it > like this. The font even says © Apple. I never really thought of it as a violation, I think I just got the font from the internet somewhere. But even if it is a violoation, it seems there is still a bug here.

Cameron Zwarich (cpst)

Comment 7 2009-02-27 08:48:50 PST

Created attachment 28072 [details] Console output of GCGetGlyphIdealMetrics errors

Cameron Zwarich (cpst)

Comment 8 2009-02-27 08:58:51 PST

Created attachment 28073 [details] Reduction Here's a simple reduction using the copy of the font on his page.

Eli Loewen

Comment 9 2009-03-06 09:12:23 PST

So has anyone made any progress on this? BTW, This font works perfectly in Gecko 1.9.1.

Cameron Zwarich (cpst)

Comment 10 2009-03-06 09:47:44 PST

Dan is probably the one to fix this bug, but since it is a regression I could try to find the point of regression.

Mark Rowe (bdash)

Comment 11 2009-03-06 09:50:36 PST

Do we have any evidence that this is a regression? The only comment about that is from me saying I don't think it is.

Cameron Zwarich (cpst)

Comment 12 2009-03-06 09:58:56 PST

(In reply to comment #11) > Do we have any evidence that this is a regression? The only comment about that > is from me saying I don't think it is. Both the site and my reduction work fine for me in Safari 3.2 Do they not work for you?

Eli Loewen

Comment 13 2009-03-06 14:43:11 PST

(In reply to comment #12) > (In reply to comment #11) > > Do we have any evidence that this is a regression? The only comment about that > > is from me saying I don't think it is. > > Both the site and my reduction work fine for me in Safari 3.2 Do they not work > for you? > It all works fine for me in Safari 3.2.

Eli Loewen

Comment 14 2009-03-10 17:21:03 PDT

Does anyone know what component this bug is related to so we are able to take it out of "New Bugs"?

Mark Rowe (bdash)

Comment 15 2009-03-10 17:22:33 PDT

There's no problem with it being in the New Bugs component.

Eli Loewen

Comment 16 2009-03-10 17:27:51 PDT

(In reply to comment #15) > There's no problem with it being in the New Bugs component. > True, its just not very specific.

mitz

Comment 17 2014-08-30 16:27:22 PDT

The bug no longer occurs (tested with Safari 7.0.6 on OS X 10.9.4).

Note You need to log in before you can comment on or make changes to this bug.