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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: components/nacl/renderer/nexe_load_manager.cc

Issue 307173002: Remove PlatformFile from components/nacl (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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
« components/nacl/browser/pnacl_host.cc ('K') | « components/nacl/renderer/nexe_load_manager.h ('k') | components/nacl/renderer/pnacl_translation_resource_host.cc » ('j') | components/nacl/renderer/ppb_nacl_private_impl.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/nacl/renderer/nexe_load_manager.cc
diff --git a/components/nacl/renderer/nexe_load_manager.cc b/components/nacl/renderer/nexe_load_manager.cc
index d2efba4cccbf955723a968a04971bbdf23da791b..0c937392230c0cb5dd4f92ed0850e61f4e67c6e0 100644
--- a/components/nacl/renderer/nexe_load_manager.cc
+++ b/components/nacl/renderer/nexe_load_manager.cc
@@ -107,7 +107,7 @@ NexeLoadManager::~NexeLoadManager() {
}
void NexeLoadManager::NexeFileDidOpen(int32_t pp_error,
- base::PlatformFile file,
+ const base::File& file,
int32_t http_status,
int64_t nexe_bytes_read,
const std::string& url,
@@ -120,7 +120,7 @@ void NexeLoadManager::NexeFileDidOpen(int32_t pp_error,
"NaCl.HttpStatusCodeClass.Nexe.NotInstalledApp",
http_status);
- if (pp_error != PP_OK || file == base::kInvalidPlatformFileValue) {
+ if (pp_error != PP_OK || !file.IsValid()) {
if (pp_error == PP_ERROR_ABORTED) {
ReportLoadAbort();
} else if (pp_error == PP_ERROR_NOACCESS) {
« components/nacl/browser/pnacl_host.cc ('K') | « components/nacl/renderer/nexe_load_manager.h ('k') | components/nacl/renderer/pnacl_translation_resource_host.cc » ('j') | components/nacl/renderer/ppb_nacl_private_impl.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698