Refactor FileSystem

content/renderer/pepper/content_renderer_pepper_host_factory.cc

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 0e7f380a8f828f2e312a57901c222cabdf4d09de..3191d31cb4dfea6a67c208b9485acb50ed77b989 100644
--- a/content/renderer/pepper/content_renderer_pepper_host_factory.cc
+++ b/content/renderer/pepper/content_renderer_pepper_host_factory.cc
@@ -10,6 +10,7 @@
 #include "content/renderer/pepper/pepper_directory_reader_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_system_host.h"
 #include "content/renderer/pepper/pepper_graphics_2d_host.h"
 #include "content/renderer/pepper/pepper_truetype_font_host.h"
 #include "content/renderer/pepper/pepper_video_capture_host.h"
@@ -45,6 +46,19 @@ scoped_ptr<ResourceHost> ContentRendererPepperHostFactory::CreateResourceHost(
 
   // Public interfaces.
   switch (message.type()) {
+    case PpapiHostMsg_FileIO_Create::ID:
+      return scoped_ptr<ResourceHost>(new PepperFileIOHost(
+          host_, instance, params.pp_resource()));
+    case PpapiHostMsg_FileSystem_Create::ID: {
+      PpapiHostMsg_FileSystem_Create::Schema::Param msg_params;
+      if (!PpapiHostMsg_FileSystem_Create::Read(&message, &msg_params)) {

[yzshen1, 2013/04/08 21:05:16]

Please use UnpackMessage() defined in ppapi_message_utils.h.

[victorhsieh, 2013/04/08 23:44:38]

Done.

+        NOTREACHED();
+        return scoped_ptr<ResourceHost>();
+      }
+      return scoped_ptr<ResourceHost>(new PepperFileSystemHost(
+          host_, instance, params.pp_resource(),
+          msg_params.a /* PP_FileSystemType */));
+    }
     case PpapiHostMsg_Graphics2D_Create::ID: {
       PpapiHostMsg_Graphics2D_Create::Schema::Param msg_params;
       if (!PpapiHostMsg_Graphics2D_Create::Read(&message, &msg_params)) {
@@ -59,9 +73,6 @@ scoped_ptr<ResourceHost> ContentRendererPepperHostFactory::CreateResourceHost(
     case PpapiHostMsg_WebSocket_Create::ID:
       return scoped_ptr<ResourceHost>(new PepperWebSocketHost(
           host_, instance, params.pp_resource()));
-    case PpapiHostMsg_FileIO_Create::ID:
-      return scoped_ptr<ResourceHost>(new PepperFileIOHost(
-          host_, instance, params.pp_resource()));
   }
 
   // Dev interfaces.