1329293 - Confusing error reported to console: "downloadable font: OS/2: adjusting Mac style (regular) ..."

Status: RESOLVED FIXED

(Core :: Graphics: Text, defect)

Description

[pushanshu]

Attachment: 9.png

User Agent: Mozilla/5.0 (Windows NT 6.2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/55.0.2883.87 Safari/537.36

Steps to reproduce:

1. Launch Firefox and open the Browser Console from menu [≡] → select Developer → select Browser Console.
2. Visit about:plugins and check for the version of Shockwave flash
3. Browse through websites that use the Flash Player plug-in and look for rendering, functional and performance issues.


Actual results:

error in console


Expected results:

no error in console

Comment 1

[Jerry Shih[:jerry] (UTC+8) (inactive)]

Is "plug-in" a better component to track this?

Comment 2

[Benjamin Smedberg]

The errors in the console don't appear to have anything to do with Flash. They are font parsing notifications from web content. I can't tell whether those are errors or warnings or what, which appears to be because the URLS are very long data: and therefore cut off the end of the message. "downloadable font: OS/2: adjusting Mac style (regular) ..."

I don't know whether we can fix that by rearranging the message so that the complete message comes before the URL, or truncating the URL, or changing the display in the error console somewhat. But I'm going to move this to fonts first because I suspect that's the better option.

http://searchfox.org/mozilla-central/rev/790b2cb423ea1ecb5746722d51633caec9bab95a/layout/style/FontFaceSet.cpp#1221

Comment 3

[Jonathan Kew [:jfkthame]]

That's a warning (rather than an error) generated by OTS, here:

http://searchfox.org/mozilla-central/rev/790b2cb423ea1ecb5746722d51633caec9bab95a/gfx/ots/src/os2.cc#152

So the font data has internal inconsistencies that OTS is fixing up, but also reporting so that the author/font developer/whatever can be alerted to the fact that their font is not quite right.

Offhand, I'm not sure where the truncation of the message happens; AFAICS, we should be getting the full URI when we call GetFamilyNameAndURIForLogging at http://searchfox.org/mozilla-central/rev/790b2cb423ea1ecb5746722d51633caec9bab95a/layout/style/FontFaceSet.cpp#1206, but it's clearly truncated in the screenshot attached. Which is probably just as well, considering that a data: URI could be huge...

Perhaps we should truncate it within gfxUserFontEntry::GetFamilyNameAndURIForLogging itself. Probably center-truncation, so that we retain the beginning and end of a potentially over-long path, would be the most useful.

Comment 4

[Jonathan Kew [:jfkthame]]

Attachment: Truncate overly long URIs in downloadable font error/warning messages

I'd suggest we just do something simple-minded like this.

Comment 5

[Jonathan Kew [:jfkthame]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/02fc9d9b46a58d20270c5dba699e3eef56370b80
Bug 1329293 - Truncate overly long URIs in downloadable font error/warning messages. r=jrmuizel

Comment 6

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/02fc9d9b46a5