Add Pepper TrueType font API plumbing.

content/browser/renderer_host/pepper/pepper_truetype_font_list_host.cc

Index: content/browser/renderer_host/pepper/pepper_truetype_font_list_host.cc
diff --git a/content/browser/renderer_host/pepper/pepper_browser_font_singleton_host.cc b/content/browser/renderer_host/pepper/pepper_truetype_font_list_host.cc
similarity index 65%
copy from content/browser/renderer_host/pepper/pepper_browser_font_singleton_host.cc
copy to content/browser/renderer_host/pepper/pepper_truetype_font_list_host.cc
index 02ae2a8754d1b844b813a7ab7d588099793ad1ed..3cc3f9a64ce6bfd56e40e9c25ac19e6937be4602 100644
--- a/content/browser/renderer_host/pepper/pepper_browser_font_singleton_host.cc
+++ b/content/browser/renderer_host/pepper/pepper_truetype_font_list_host.cc
@@ -1,15 +1,17 @@
-// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Copyright (c) 2013 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 "content/browser/renderer_host/pepper/pepper_browser_font_singleton_host.h"
+#include "content/browser/renderer_host/pepper/pepper_truetype_font_list_host.h"
+
+#include <algorithm>
 
 #include "base/threading/sequenced_worker_pool.h"
-#include "base/values.h"
-#include "content/common/font_list.h"
+#include "content/browser/renderer_host/pepper/pepper_truetype_font_list.h"
 #include "content/public/browser/browser_ppapi_host.h"
 #include "content/public/browser/browser_thread.h"
 #include "ppapi/host/dispatch_host_message.h"
+#include "ppapi/host/host_message_context.h"
 #include "ppapi/host/resource_message_filter.h"
 #include "ppapi/proxy/ppapi_messages.h"
 
@@ -56,7 +58,7 @@ int32_t FontMessageFilter::OnResourceMessageReceived(
     ppapi::host::HostMessageContext* context) {
   IPC_BEGIN_MESSAGE_MAP(FontMessageFilter, msg)
     PPAPI_DISPATCH_HOST_RESOURCE_CALL_0(
-        PpapiHostMsg_BrowserFontSingleton_GetFontFamilies,
+        PpapiHostMsg_TrueTypeFontSingleton_GetFontFamilies,
         OnHostMsgGetFontFamilies)
   IPC_END_MESSAGE_MAP()
   return PP_ERROR_FAILED;
@@ -65,35 +67,23 @@ int32_t FontMessageFilter::OnResourceMessageReceived(
 int32_t FontMessageFilter::OnHostMsgGetFontFamilies(
     ppapi::host::HostMessageContext* context) {
   // OK to use "slow blocking" version since we're on the blocking pool.
-  scoped_ptr<base::ListValue> list(GetFontList_SlowBlocking());
-
-  std::string output;
-  for (size_t i = 0; i < list->GetSize(); i++) {
-    base::ListValue* cur_font;
-    if (!list->GetList(i, &cur_font))
-      continue;
-
-    // Each entry is actually a list of (font name, localized name).
-    // We only care about the regular name.
-    std::string font_name;
-    if (!cur_font->GetString(0, &font_name))
-      continue;
-
-    // Font names are separated with nulls. We also want an explicit null at
-    // the end of the string (Pepper strings aren't null terminated so since
-    // we specify there will be a null, it should actually be in the string).
-    output.append(font_name);
-    output.push_back(0);
-  }
-
+  std::vector<std::string> font_families;
+  GetFontFamilies_SlowBlocking(&font_families);
+  // Sort the names in case the host platform returns them out of order.
+  std::sort(font_families.begin(), font_families.end());
+
+  int result = static_cast<int32_t>(font_families.size());

[jschuh, 2013/03/14 21:56:07]

Use a checked_numeric_cast here.

[bbudge, 2013/03/14 23:20:23]

Done.

+  ppapi::host::ReplyMessageContext reply_context =
+      context->MakeReplyMessageContext();
+  reply_context.params.set_result(result);
   context->reply_msg =
-      PpapiPluginMsg_BrowserFontSingleton_GetFontFamiliesReply(output);
-  return PP_OK;
+      PpapiPluginMsg_TrueTypeFontSingleton_GetFontFamiliesReply(font_families);
+  return result;
 }
 
 }  // namespace
 
-PepperBrowserFontSingletonHost::PepperBrowserFontSingletonHost(
+PepperTrueTypeFontListHost::PepperTrueTypeFontListHost(
     BrowserPpapiHost* host,
     PP_Instance instance,
     PP_Resource resource)
@@ -102,7 +92,7 @@ PepperBrowserFontSingletonHost::PepperBrowserFontSingletonHost(
       new FontMessageFilter()));
 }
 
-PepperBrowserFontSingletonHost::~PepperBrowserFontSingletonHost() {
+PepperTrueTypeFontListHost::~PepperTrueTypeFontListHost() {
 }
 
 }  // namespace content