Use the WebKit default fonts when specifying generic font families.

content/renderer/pepper_plugin_delegate_impl.cc

Index: content/renderer/pepper_plugin_delegate_impl.cc
===================================================================
--- content/renderer/pepper_plugin_delegate_impl.cc	(revision 86427)
+++ content/renderer/pepper_plugin_delegate_impl.cc	(working copy)
@@ -45,6 +45,7 @@
 #include "ppapi/c/private/ppb_flash_net_connector.h"
 #include "ppapi/proxy/host_dispatcher.h"
 #include "ppapi/proxy/ppapi_messages.h"
+#include "ppapi/shared_impl/ppapi_preferences.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFileChooserCompletion.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebFileChooserParams.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebPluginContainer.h"
@@ -328,7 +329,8 @@
   DispatcherWrapper() {}
   virtual ~DispatcherWrapper() {}
 
-  bool Init(base::ProcessHandle plugin_process_handle,
+  bool Init(RenderView* render_view,
+            base::ProcessHandle plugin_process_handle,
             const IPC::ChannelHandle& channel_handle,
             PP_Module pp_module,
             pp::proxy::Dispatcher::GetInterfaceFunc local_get_interface);
@@ -351,6 +353,7 @@
 }  // namespace
 
 bool DispatcherWrapper::Init(
+    RenderView* render_view,
     base::ProcessHandle plugin_process_handle,
     const IPC::ChannelHandle& channel_handle,
     PP_Module pp_module,
@@ -358,8 +361,10 @@
   dispatcher_.reset(new pp::proxy::HostDispatcher(
       plugin_process_handle, pp_module, local_get_interface));
 
-  if (!dispatcher_->InitHostWithChannel(PepperPluginRegistry::GetInstance(),
-                                        channel_handle, true)) {
+  if (!dispatcher_->InitHostWithChannel(
+          PepperPluginRegistry::GetInstance(),
+          channel_handle, true,
+          ppapi::Preferences(render_view->webkit_preferences()))) {
     dispatcher_.reset();
     return false;
   }
@@ -608,6 +613,7 @@
   PepperPluginRegistry::GetInstance()->AddLiveModule(path, module);
   scoped_ptr<DispatcherWrapper> dispatcher(new DispatcherWrapper);
   if (!dispatcher->Init(
+          render_view_,
           plugin_process_handle, channel_handle,
           module->pp_module(),
           webkit::ppapi::PluginModule::GetLocalGetInterfaceFunc()))
@@ -1319,3 +1325,7 @@
   shm.reset(new base::SharedMemory(handle, false));
   return shm.release();
 }
+
+::ppapi::Preferences PepperPluginDelegateImpl::GetPreferences() {
+  return ::ppapi::Preferences(render_view_->webkit_preferences());

[yzshen1, 2011/05/25 23:40:11]

I think here the '::' prefix is not mandatory, right?
If it is here just for readability, it might be good to keep consistent and
change other occurrences such as line 367.

+}