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

Unified Diff: ppapi/proxy/ppb_file_ref_proxy.cc

Issue 13726024: Refactor FileSystem (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 8 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: ppapi/proxy/ppb_file_ref_proxy.cc
diff --git a/ppapi/proxy/ppb_file_ref_proxy.cc b/ppapi/proxy/ppb_file_ref_proxy.cc
index c63cccd1d083ba89fa9084ce221eb4065baa0b7f..87cb16ee58f2d320648437ee6fca0c1b6babd858 100644
--- a/ppapi/proxy/ppb_file_ref_proxy.cc
+++ b/ppapi/proxy/ppb_file_ref_proxy.cc
@@ -204,18 +204,13 @@ PPB_FileRef_Proxy::~PPB_FileRef_Proxy() {
}
// static
-PP_Resource PPB_FileRef_Proxy::CreateProxyResource(PP_Resource file_system,
+PP_Resource PPB_FileRef_Proxy::CreateProxyResource(PP_Instance instance,
+ PP_Resource file_system,
const char* path) {
- Resource* file_system_object =
- PpapiGlobals::Get()->GetResourceTracker()->GetResource(file_system);
- if (!file_system_object)
- return 0;
-
PPB_FileRef_CreateInfo create_info;
- PluginDispatcher::GetForResource(file_system_object)->Send(
+ PluginDispatcher::GetForInstance(instance)->Send(
new PpapiHostMsg_PPBFileRef_Create(
- API_ID_PPB_FILE_REF, file_system_object->host_resource(),
- path, &create_info));
+ API_ID_PPB_FILE_REF, instance, file_system, path, &create_info));
return PPB_FileRef_Proxy::DeserializeFileRef(create_info);
}
@@ -261,14 +256,16 @@ PP_Resource PPB_FileRef_Proxy::DeserializeFileRef(
}
#if !defined(OS_NACL)
-void PPB_FileRef_Proxy::OnMsgCreate(const HostResource& file_system,
+void PPB_FileRef_Proxy::OnMsgCreate(PP_Instance pp_instance,
+ PP_Resource file_system,
const std::string& path,
PPB_FileRef_CreateInfo* result) {
- thunk::EnterResourceCreation enter(file_system.instance());
+ thunk::EnterResourceCreation enter(pp_instance);
if (enter.failed())
return;
+
PP_Resource resource = enter.functions()->CreateFileRef(
- file_system.host_resource(), path.c_str());
+ pp_instance, file_system, path.c_str());
if (!resource)
return; // CreateInfo default constructor initializes to 0.
SerializeFileRef(resource, result);

Powered by Google App Engine
This is Rietveld 408576698