Refactor BrowserFont to new design

content/renderer/pepper/pepper_in_process_resource_creation.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 "content/renderer/pepper/pepper_in_process_resource_creation.h"
 
 #include "base/bind.h"
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop.h"
 #include "content/renderer/render_view_impl.h"
 #include "content/renderer/pepper/pepper_in_process_router.h"
 #include "content/renderer/pepper/renderer_ppapi_host_impl.h"
 #include "ipc/ipc_message.h"
 #include "ipc/ipc_message_macros.h"
 #include "ppapi/host/ppapi_host.h"
+#include "ppapi/proxy/browser_font_resource_trusted.h"
 #include "ppapi/proxy/file_chooser_resource.h"
 #include "ppapi/proxy/ppapi_messages.h"
 #include "ppapi/proxy/printing_resource.h"
 #include "ppapi/proxy/url_request_info_resource.h"
 #include "ppapi/proxy/url_response_info_resource.h"
 #include "ppapi/proxy/websocket_resource.h"
 #include "ppapi/shared_impl/ppapi_globals.h"
 #include "ppapi/shared_impl/ppapi_permissions.h"
 #include "ppapi/shared_impl/resource_tracker.h"
 #include "webkit/plugins/ppapi/ppapi_plugin_instance.h"
 
 // Note that the code in the creation functions in this file should generally
 // be the same as that in ppapi/proxy/resource_creation_proxy.cc. See
 // pepper_in_process_resource_creation.h for what this file is for.
 
 namespace content {
 
 // PepperInProcessResourceCreation --------------------------------------------
 
 PepperInProcessResourceCreation::PepperInProcessResourceCreation(
     RendererPpapiHostImpl* host_impl,
     webkit::ppapi::PluginInstance* instance)
     : ResourceCreationImpl(instance),
       host_impl_(host_impl) {
 }
 
 PepperInProcessResourceCreation::~PepperInProcessResourceCreation() {
 }
 
+PP_Resource PepperInProcessResourceCreation::CreateBrowserFont(
+    PP_Instance instance,
+    const PP_BrowserFont_Trusted_Description* description) {
+  ppapi::Preferences prefs(
+      host_impl_->GetRenderViewForInstance(instance)->GetWebkitPreferences());
+  scoped_refptr<ppapi::proxy::BrowserFontResource_Trusted> font(
+      new ppapi::proxy::BrowserFontResource_Trusted(
+          host_impl_->in_process_router()->GetPluginConnection(),
+          instance,
+          *description,
+          prefs));
+  if (!ppapi::proxy::BrowserFontResource_Trusted::IsPPFontDescriptionValid(

[brettw, 2012/11/26 23:50:37]

Can you do this at the beginning of the function?

[victorhsieh, 2012/11/27 03:00:50]

Done.

+      *description))
+    return 0;
+  return font->GetReference();
+}
+
 PP_Resource PepperInProcessResourceCreation::CreateFileChooser(
     PP_Instance instance,
     PP_FileChooserMode_Dev mode,
     const char* accept_types) {
   return (new ppapi::proxy::FileChooserResource(
       host_impl_->in_process_router()->GetPluginConnection(),
       instance, mode, accept_types))->GetReference();
 }
 
 PP_Resource PepperInProcessResourceCreation::CreatePrinting(
@@ skipping 21 matching lines @@
 }
 
 PP_Resource PepperInProcessResourceCreation::CreateWebSocket(
     PP_Instance instance) {
   return (new ppapi::proxy::WebSocketResource(
       host_impl_->in_process_router()->GetPluginConnection(),
       instance))->GetReference();
 }
 
 }  // namespace content