Bug 34322 – Web Inspector: REGRESSION(r54001): Web inspector monospace font is wrong on some versions of Mac OS X

• Status: RESOLVED
• Resolution: FIXED
• Component: Web Inspector (Deprecated)

[Mark Rowe (bdash)]

On Mac OS X 10.6 the monospace font in the inspector is Monaco rather than Menlo as expected.

This regression was caused by <http://trac.webkit.org/changeset/54001>, which introduced JavaScript code to detect that Mac OS X version and uses that information to choose which monospace font should be used.  However, the manner in which it parses the Mac OS X version is broken.  It assumes that Mac OS X version numbers always have three components, which is clearly not the case.  The code also makes the strange decision that if the user agent string is from a version of Mac OS X it is not familiar with that it must be running on 10.5.

It’s not at all clear to me why the inspector is attempting to detect platforms from JavaScript like this.  The inspector was already set up to use the correct platform-specific monospace fonts by configuring the default fixed-width font and size before creating the inspector’s WebView (see <http://trac.webkit.org/browser/trunk/WebKit/mac/WebCoreSupport/WebInspectorClient.mm#L189> for example).

[Mark Rowe (bdash)]

<rdar://problem/7590854>

[Alexander Pavlov (apavlov)]

Created attachment 47716 [details]
[PATCH] Fix MacOS X version detection (in userAgent can have 2 or 3 segments)

[WebKit Commit Bot]

Comment on attachment 47716 [details]
[PATCH] Fix MacOS X version detection (in userAgent can have 2 or 3 segments)

Clearing flags on attachment: 47716

Committed r54067: <http://trac.webkit.org/changeset/54067>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.