Add getFontList() to Font Settings Extension API.

chrome/browser/extensions/extension_font_settings_api.cc

 // Copyright (c) 2012 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/extensions/extension_font_settings_api.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/stringprintf.h"
 #include "base/values.h"
 #include "chrome/browser/extensions/extension_preference_helpers.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/common/extensions/extension_error_utils.h"
+#include "content/public/browser/font_list_async.h"
 
 namespace {
 
 const char kGenericFamilyKey[] = "genericFamily";
 const char kFontNameKey[] = "fontName";
+const char kLocalizedNameKey[] = "localizedName";
 const char kScriptKey[] = "script";
 
 // Format for per-script font preference keys.
 // E.g., "webkit.webprefs.fonts.standard.Hrkt"
 const char kWebKitPerScriptFontPrefFormat[] = "webkit.webprefs.fonts.%s.%s";
 
 // Format for global (non per-script) font preference keys.
 // E.g., "webkit.webprefs.global.fixed_font_family"
 // Note: there are two meanings of "global" here. The "Global" in the const name
 // means "not per-script". The "global" in the key itself means "not per-tab"
 // (per-profile).
 const char kWebKitGlobalFontPrefFormat[] =
     "webkit.webprefs.global.%s_font_family";
 
 // Gets the font name preference path from |details| which contains key
 // |kGenericFamilyKey| and optionally |kScriptKey|.
 bool GetFontNamePrefPath(DictionaryValue* details, std::string* pref_path)
 {
   std::string generic_family;
   if (!details->GetString(kGenericFamilyKey, &generic_family))
     return false;
 
   if (details->HasKey(kScriptKey)) {
     std::string script;
     if (!details->GetString(kScriptKey, &script))
       return false;
-    *pref_path = base::StringPrintf(kWebKitPerScriptFontPrefFormat,
+    *pref_path = StringPrintf(kWebKitPerScriptFontPrefFormat,
                                    generic_family.c_str(),
                                    script.c_str());
   } else {
-    *pref_path = base::StringPrintf(kWebKitGlobalFontPrefFormat,
+    *pref_path = StringPrintf(kWebKitGlobalFontPrefFormat,
                                    generic_family.c_str());
   }
 
   return true;
 }
 
 }  // namespace
 
 bool GetFontNameFunction::RunImpl() {
   DictionaryValue* details = NULL;
@@ skipping 25 matching lines @@
   std::string font_name;
   EXTENSION_FUNCTION_VALIDATE(details->GetString(kFontNameKey, &font_name));
 
   // Ensure |pref_path| really is for a registered per-script font pref.
   EXTENSION_FUNCTION_VALIDATE(
       profile_->GetPrefs()->FindPreference(pref_path.c_str()));
 
   ExtensionPrefs* prefs = profile_->GetExtensionService()->extension_prefs();
   prefs->SetExtensionControlledPref(extension_id(), pref_path.c_str(),
                                     kExtensionPrefsScopeRegular,
-                                    base::Value::CreateStringValue(font_name));
+                                    Value::CreateStringValue(font_name));
   return true;
 }
+
+bool GetFontListFunction::RunImpl() {
+  content::GetFontListAsync(
+      Bind(&GetFontListFunction::FontListHasLoaded, this));
+  return true;
+}
+
+void GetFontListFunction::FontListHasLoaded(scoped_ptr<ListValue> list) {
+  bool success = CopyFontsToResult(list.get());
+  SendResponse(success);
+}
+
+bool GetFontListFunction::CopyFontsToResult(ListValue* fonts)
+{
+  ListValue* result = new ListValue();

[Matt Perry, 2012/03/01 22:57:29]

Use a scoped_ptr to avoid leaking result.

[falken, 2012/03/01 23:30:52]

Done.

+  for (ListValue::iterator it = fonts->begin(); it != fonts->end(); ++it) {
+    ListValue *fontListValue;

[Matt Perry, 2012/03/01 22:57:29]

ListValue* font_list_value

[falken, 2012/03/01 23:30:52]

Done.

+    if (!(*it)->GetAsList(&fontListValue)) {
+      NOTREACHED();
+      return false;
+    }
+
+    std::string name;
+    if (!fontListValue->GetString(0, &name)) {
+      NOTREACHED();
+      return false;
+    }
+
+    std::string localizedName;

[Matt Perry, 2012/03/01 22:57:29]

naming style

[falken, 2012/03/01 23:30:52]

Done.

+    if (!fontListValue->GetString(1, &localizedName)) {
+      NOTREACHED();
+      return false;
+    }
+
+    DictionaryValue* fontName = new DictionaryValue();

[Matt Perry, 2012/03/01 22:57:29]

ditto

[falken, 2012/03/01 23:30:52]

Done.

+    fontName->Set(kFontNameKey, Value::CreateStringValue(name));
+    fontName->Set(kLocalizedNameKey, Value::CreateStringValue(localizedName));
+    result->Append(fontName);
+  }
+
+  result_.reset(result);
+  return true;
+}