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

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
« no previous file with comments | « content/renderer/pepper/ppb_file_ref_impl.cc ('k') | ppapi/proxy/ppapi_messages.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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..1387eb7c58b9bdcbdb2fa5ab3007a7cc45bc41c1 100644
--- a/ppapi/proxy/flash_file_resource.cc
+++ b/ppapi/proxy/flash_file_resource.cc
@@ -176,9 +176,8 @@ 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) ||
+ !ppapi::PepperFileOpenFlagsToPlatformFileFlags(mode, NULL) ||
!file)
return PP_ERROR_BADARGUMENT;
@@ -187,7 +186,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;
« no previous file with comments | « content/renderer/pepper/ppb_file_ref_impl.cc ('k') | ppapi/proxy/ppapi_messages.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698