Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(5454)

Unified Diff: chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc

Issue 10905227: Introduce PPB_Flash_Font. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 8 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc
diff --git a/chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc b/chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc
index ebdf6ba25cda583b9eff11ed022b56f16b9f4cbb..f40d43deaf91a4a42795bb609d2b603327c09159 100644
--- a/chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc
+++ b/chrome/renderer/pepper/chrome_renderer_pepper_host_factory.cc
@@ -4,14 +4,21 @@
#include "chrome/renderer/pepper/chrome_renderer_pepper_host_factory.h"
+#include "base/logging.h"
+#include "chrome/renderer/pepper/pepper_flash_font_file_host.h"
+#include "content/public/renderer/renderer_ppapi_host.h"
+#include "ppapi/host/ppapi_host.h"
#include "ppapi/host/resource_host.h"
#include "ppapi/proxy/ppapi_messages.h"
+#include "ppapi/shared_impl/ppapi_permissions.h"
using ppapi::host::ResourceHost;
namespace chrome {
-ChromeRendererPepperHostFactory::ChromeRendererPepperHostFactory() {
+ChromeRendererPepperHostFactory::ChromeRendererPepperHostFactory(
+ content::RendererPpapiHost* host)
+ : host_(host) {
}
ChromeRendererPepperHostFactory::~ChromeRendererPepperHostFactory() {
@@ -23,7 +30,25 @@ ChromeRendererPepperHostFactory::CreateResourceHost(
const ppapi::proxy::ResourceMessageCallParams& params,
PP_Instance instance,
const IPC::Message& message) {
- // There are no Chrome-side implementations of resources.
+ DCHECK(host == host_->GetPpapiHost());
+
+ // Make sure the plugin is giving us a valid instance for this resource.
+ if (!host_->IsValidInstance(instance))
+ return scoped_ptr<ResourceHost>();
+
+ if (host_->GetPpapiHost()->permissions().HasPermission(
+ ppapi::PERMISSION_FLASH)) {
+ switch (message.type()) {
+ case PpapiHostMsg_FlashFontFile_Create::ID:
+ PpapiHostMsg_FlashFontFile_Create::Param param;
+ if (PpapiHostMsg_FlashFontFile_Create::Read(&message, &param)) {
+ return scoped_ptr<ResourceHost>(new PepperFlashFontFileHost(
+ host_, instance, params.pp_resource(), param.a, param.b));
+ }
+ break;
+ }
+ }
+
return scoped_ptr<ResourceHost>();
}
« no previous file with comments | « chrome/renderer/pepper/chrome_renderer_pepper_host_factory.h ('k') | chrome/renderer/pepper/pepper_flash_font_file_host.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698