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

Unified Diff: content/renderer/pepper/content_renderer_pepper_host_factory.cc

Issue 21966004: Pepper: Move FileRef to the "new" resource proxy. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Build fix Created 7 years, 3 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: content/renderer/pepper/content_renderer_pepper_host_factory.cc
diff --git a/content/renderer/pepper/content_renderer_pepper_host_factory.cc b/content/renderer/pepper/content_renderer_pepper_host_factory.cc
index 983e18526df69b7cd8d62555910a0093073125f2..fd446377b4f03b4e563c7f7d4d7e7b0d0ce64b9d 100644
--- a/content/renderer/pepper/content_renderer_pepper_host_factory.cc
+++ b/content/renderer/pepper/content_renderer_pepper_host_factory.cc
@@ -11,6 +11,7 @@
#include "content/renderer/pepper/pepper_audio_input_host.h"
#include "content/renderer/pepper/pepper_file_chooser_host.h"
#include "content/renderer/pepper/pepper_file_io_host.h"
+#include "content/renderer/pepper/pepper_file_ref_renderer_host.h"
#include "content/renderer/pepper/pepper_file_system_host.h"
#include "content/renderer/pepper/pepper_graphics_2d_host.h"
#include "content/renderer/pepper/pepper_truetype_font_host.h"
@@ -75,6 +76,17 @@ scoped_ptr<ResourceHost> ContentRendererPepperHostFactory::CreateResourceHost(
case PpapiHostMsg_FileIO_Create::ID:
return scoped_ptr<ResourceHost>(new PepperFileIOHost(
host_, instance, params.pp_resource()));
+ case PpapiHostMsg_FileRef_CreateInternal::ID: {
+ PP_Resource file_system;
+ std::string internal_path;
+ if (!UnpackMessage<PpapiHostMsg_FileRef_CreateInternal>(
+ message, &file_system, &internal_path)) {
dmichael (off chromium) 2013/09/03 22:34:50 nit: should this be indented another 4?
teravest 2013/09/04 14:46:25 I don't think so. However, I'm able to barely fit
dmichael (off chromium) 2013/09/04 17:23:54 That works for me. (But if that hadn't worked, I t
+ NOTREACHED();
+ return scoped_ptr<ResourceHost>();
+ }
+ return scoped_ptr<ResourceHost>(new PepperFileRefRendererHost(
+ host_, instance, params.pp_resource(), file_system, internal_path));
+ }
case PpapiHostMsg_FileSystem_Create::ID: {
PP_FileSystemType file_system_type;
if (!UnpackMessage<PpapiHostMsg_FileSystem_Create>(message,

Powered by Google App Engine
This is Rietveld 408576698