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

Unified Diff: ppapi/api/private/ppb_nacl_private.idl

Issue 231243002: PPAPI: Move some of NexeFileDidOpen to NexeLoadManager (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix crash Created 6 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
« no previous file with comments | « components/nacl/renderer/ppb_nacl_private_impl.cc ('k') | ppapi/c/private/ppb_nacl_private.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ppapi/api/private/ppb_nacl_private.idl
diff --git a/ppapi/api/private/ppb_nacl_private.idl b/ppapi/api/private/ppb_nacl_private.idl
index cb2b1b7d9e8fb4975bffa7fedbbf04f32b3f975a..c27fc306cb8b6d420dc44c741b2ef6780ef481fa 100644
--- a/ppapi/api/private/ppb_nacl_private.idl
+++ b/ppapi/api/private/ppb_nacl_private.idl
@@ -271,6 +271,18 @@ interface PPB_NaCl_Private {
[in] uint64_t loaded_bytes,
[in] uint64_t total_bytes);
+ /* Report that the attempt to open the nexe has finished. Opening the file
+ * may have failed, as indicated by a pp_error value that is not PP_OK or an
+ * fd of -1. Failure to stat the file to determine its length results in
+ * nexe_bytes_read being -1.
+ */
+ void NexeFileDidOpen([in] PP_Instance instance,
+ [in] int32_t pp_error,
+ [in] int32_t fd,
+ [in] int32_t http_status,
+ [in] int64_t nexe_bytes_read,
+ [in] str_t url);
+
/* Report that the nexe loaded successfully. */
void ReportLoadSuccess([in] PP_Instance instance,
[in] str_t url,
« no previous file with comments | « components/nacl/renderer/ppb_nacl_private_impl.cc ('k') | ppapi/c/private/ppb_nacl_private.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698