Index: content/renderer/pepper/pepper_file_io_host.cc |
diff --git a/content/renderer/pepper/pepper_file_io_host.cc b/content/renderer/pepper/pepper_file_io_host.cc |
index 4750f8f797e3c5541b42bb05f1b5193de14a2ef7..96b5f6b7cbe98ad2ec3d7f6a63538a3599f32ed6 100644 |
--- a/content/renderer/pepper/pepper_file_io_host.cc |
+++ b/content/renderer/pepper/pepper_file_io_host.cc |
@@ -194,10 +194,14 @@ int32_t PepperFileIOHost::OnHostMsgOpen( |
if (rv != PP_OK) |
return rv; |
- int flags = 0; |
+ // TODO(tommycli): Eventually just pass the Pepper flags straight to the |
+ // FileSystemDispatcher so it can handle doing the security check. |
+ int platform_file_flags = 0; |
open_flags_ = open_flags; |
- if (!::ppapi::PepperFileOpenFlagsToPlatformFileFlags(open_flags, &flags)) |
+ if (!::ppapi::PepperFileOpenFlagsToPlatformFileFlags(open_flags, |
+ &platform_file_flags)) { |
return PP_ERROR_BADARGUMENT; |
+ } |
EnterResourceNoLock<PPB_FileRef_API> enter(file_ref_resource, true); |
if (enter.failed()) |
@@ -223,7 +227,7 @@ int32_t PepperFileIOHost::OnHostMsgOpen( |
weak_factory_.GetWeakPtr(), |
context->MakeReplyMessageContext()); |
file_system_dispatcher->OpenFile( |
- file_system_url_, flags, |
+ file_system_url_, platform_file_flags, |
base::Bind(&DidOpenFileSystemURL, callback), |
base::Bind(&DidFailOpenFileSystemURL, callback)); |
} else { |
@@ -232,7 +236,7 @@ int32_t PepperFileIOHost::OnHostMsgOpen( |
if (file_system_type_ != PP_FILESYSTEMTYPE_EXTERNAL || !helper) |
return PP_ERROR_FAILED; |
if (!helper->AsyncOpenFile( |
- file_ref->GetSystemPath(), flags, |
+ file_ref->GetSystemPath(), open_flags, |
base::Bind(&PepperFileIOHost::ExecutePlatformOpenFileCallback, |
weak_factory_.GetWeakPtr(), |
context->MakeReplyMessageContext()))) |