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

Unified Diff: ppapi/proxy/flash_file_resource.cc

Issue 19723010: Pepper Message Filters: Port to use explicit permission grants in ChildProcessSecurityPolicy. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@0044-write-support-remove-child-process-security-policy-bitmask-usage
Patch Set: Created 7 years, 5 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/flash_file_resource.cc
diff --git a/ppapi/proxy/flash_file_resource.cc b/ppapi/proxy/flash_file_resource.cc
index a7db4c00a836ddbd25d586161508d0e38ef73c91..b0a0987736464ea5671c321348965a705fe01f6e 100644
--- a/ppapi/proxy/flash_file_resource.cc
+++ b/ppapi/proxy/flash_file_resource.cc
@@ -176,10 +176,7 @@ int32_t FlashFileResource::OpenFileHelper(const std::string& path,
PepperFilePath::Domain domain_type,
int32_t mode,
PP_FileHandle* file) {
- int flags = 0;
- if (path.empty() ||
- !ppapi::PepperFileOpenFlagsToPlatformFileFlags(mode, &flags) ||
- !file)
+ if (path.empty() || !file)
return PP_ERROR_BADARGUMENT;
PepperFilePath pepper_path(domain_type, base::FilePath::FromUTF8Unsafe(path));
@@ -187,7 +184,7 @@ int32_t FlashFileResource::OpenFileHelper(const std::string& path,
IPC::Message unused;
ResourceMessageReplyParams reply_params;
int32_t error = GenericSyncCall(BROWSER,
- PpapiHostMsg_FlashFile_OpenFile(pepper_path, flags), &unused,
+ PpapiHostMsg_FlashFile_OpenFile(pepper_path, mode), &unused,
&reply_params);
if (error != PP_OK)
return error;

Powered by Google App Engine
This is Rietveld 408576698