Add a load_status callback hook that is invoked at the end of chrome LoadApp.

src/public/chrome_main.h

Index: src/public/chrome_main.h
diff --git a/src/public/chrome_main.h b/src/public/chrome_main.h
index 96885b38edbe2b76c70ded780857d9b86da83886..09ec5c7c48fa8c917a8ce43528186ef035d21c5f 100644
--- a/src/public/chrome_main.h
+++ b/src/public/chrome_main.h
@@ -162,6 +162,13 @@ struct NaClChromeMainArgs {
                                              size_t info_size);
 #endif
 
+  /*
+   * Callback to run when the initial module load status from a call to
+   * NaClChromeMainStart is known (load_status is a value from the

[Mark Seaborn, 2015/04/18 01:14:14]

Can you also say that it's called from the thread that called
NaClChromeMainStart()?

[jvoung (off chromium), 2015/04/20 18:49:47]

Good idea -- done.

+   * NaClErrorCode enumeration and may indicate success or error).

[Mark Seaborn, 2015/04/18 01:14:14]

Technically NaClErrorCode is not defined in src/public/ yet, so can you say:

load_status is zero on success, or a non-zero error code (from the NaClErrorCode
enumeration) on failure.

This way, at least success vs. failure can be determined via the src/public/
interface. :-)

[jvoung (off chromium), 2015/04/20 18:49:47]

Done.

+   */

[Mark Seaborn, 2015/04/18 01:14:14]

Add: "Optional; may be NULL" to follow the style of the other comments.

[jvoung (off chromium), 2015/04/20 18:49:47]

Done.

+  void (*load_status_handler_func)(int load_status);
+
 #if NACL_LINUX || NACL_OSX
   /*
    * The result of sysconf(_SC_NPROCESSORS_ONLN).  The Chrome