Pepper: Move FileRef to the "new" resource proxy.

ppapi/thunk/ppb_file_ref_thunk.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 "ppapi/c/pp_file_info.h"
 #include "ppapi/c/ppb_file_ref.h"
 #include "ppapi/c/pp_completion_callback.h"
 #include "ppapi/c/pp_errors.h"
 #include "ppapi/c/private/ppb_file_ref_private.h"
+#include "ppapi/shared_impl/file_ref_create_info.h"
 #include "ppapi/shared_impl/proxy_lock.h"
 #include "ppapi/shared_impl/tracked_callback.h"
 #include "ppapi/thunk/enter.h"
 #include "ppapi/thunk/thunk.h"
 #include "ppapi/thunk/ppb_file_ref_api.h"
 #include "ppapi/thunk/ppb_file_system_api.h"
 #include "ppapi/thunk/resource_creation_api.h"
 
 namespace ppapi {
 namespace thunk {
 
 namespace {
 
 typedef EnterResource<PPB_FileRef_API> EnterFileRef;
 
 PP_Resource Create(PP_Resource file_system, const char* path) {
   VLOG(4) << "PPB_FileRef::Create()";
   ppapi::ProxyAutoLock lock;
   EnterResourceNoLock<PPB_FileSystem_API> enter_file_system(file_system, true);
   if (enter_file_system.failed())
     return 0;
   PP_Instance instance = enter_file_system.resource()->pp_instance();
   EnterResourceCreationNoLock enter(instance);
   if (enter.failed())
     return 0;
-  return enter.functions()->CreateFileRef(instance, file_system, path);
+  FileRef_CreateInfo info;
+  info.file_system_type = enter_file_system.object()->GetType();
+  info.internal_path = std::string(path);
+  info.pending_host_resource_id = 0;
+  info.file_system_plugin_resource = file_system;

[dmichael (off chromium), 2013/08/07 22:19:13]

seems like a FileRefCreateInfo constructor would be nice?

[teravest, 2013/08/08 00:50:06]

This is the only place where we have to mess with it much. If you want, I can
add one, though.

+  return enter.functions()->CreateFileRef(instance, info);
 }
 
 PP_Bool IsFileRef(PP_Resource resource) {
   VLOG(4) << "PPB_FileRef::IsFileRef()";
   EnterFileRef enter(resource, false);
   return PP_FromBool(enter.succeeded());
 }
 
 PP_FileSystemType GetFileSystemType(PP_Resource file_ref) {
   VLOG(4) << "PPB_FileRef::GetFileSystemType()";
@@ skipping 140 matching lines @@
 const PPB_FileRef_1_1* GetPPB_FileRef_1_1_Thunk() {
   return &g_ppb_file_ref_thunk_1_1;
 }
 
 const PPB_FileRefPrivate_0_1* GetPPB_FileRefPrivate_0_1_Thunk() {
   return &g_ppb_file_ref_private_thunk;
 }
 
 }  // namespace thunk
 }  // namespace ppapi