Changed fontlist to list all avalilable fonts.

chrome/browser/ui/webui/options/font_settings_utils_mac.mm

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/webui/options/font_settings_utils.h"
 
 #import <Cocoa/Cocoa.h>
 
 #include "base/mac/scoped_nsautorelease_pool.h"
 #include "base/sys_string_conversions.h"
 #include "base/values.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/common/pref_names.h"
 
 static void ValidateFontFamily(PrefService* prefs,
                                const char* family_pref_name) {
   // The native font settings dialog saved fonts by the font name, rather
   // than the family name.  This worked for the old dialog since
   // -[NSFont fontWithName:size] accepted a font or family name, but the
   // behavior was technically wrong.  Since we really need the family name for
   // the dom-ui options window, we will fix the saved preference if necessary.
-  NSString *family_name =
+  NSString *pref_value =
       base::SysUTF8ToNSString(prefs->GetString(family_pref_name));
-  NSFont *font = [NSFont fontWithName:family_name
+  NSFont *font = [NSFont fontWithName:pref_value
                                  size:[NSFont systemFontSize]];
+  NSString* familyName = [font familyName];

[brettw, 2011/06/24 20:56:26]

Local variables should be lower_case_with_underscores.

+  NSFontManager* fontManager = [[[NSFontManager alloc] init] autorelease];

[csilv, 2011/06/24 20:51:30]

Use scoped_nsobject instead of autorelease.

   if (font &&
-      [[font familyName] caseInsensitiveCompare:family_name] != NSOrderedSame) {
-    std::string new_family_name = base::SysNSStringToUTF8([font familyName]);
-    prefs->SetString(family_pref_name, new_family_name);
+      [familyName caseInsensitiveCompare:pref_value] != NSOrderedSame &&
+      // The condition below is for Osaka-Mono which exceptionally stores font

[brettw, 2011/06/24 20:56:26]

I'd probably not put the comment in the middle of the if statement, and instead
put it above, with "The fixed pitch check is for..."

+      // name instead of font family name. See GetFontList_SlowBlocking
+      // in font_list_mac.mm for more info.
+      [fontManager fontNamed:familyName hasTraits:NSFixedPitchFontMask] ==
+      [fontManager fontNamed:pref_value hasTraits:NSFixedPitchFontMask]) {
+    std::string new_pref_value = base::SysNSStringToUTF8(familyName);
+    prefs->SetString(family_pref_name, new_pref_value);
   }
 }
 
 // static
 void FontSettingsUtilities::ValidateSavedFonts(PrefService* prefs) {
   ValidateFontFamily(prefs, prefs::kWebKitSerifFontFamily);
   ValidateFontFamily(prefs, prefs::kWebKitSansSerifFontFamily);
   ValidateFontFamily(prefs, prefs::kWebKitFixedFontFamily);
 }