Add success status to ReportTranslationFinished in ppb_nacl_private and IPC

ppapi/api/private/ppb_nacl_private.idl

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 1054723c85cb6badc7c54766ee130e0f4de1ac77..0a5084997ecc5f00477561d38455f3c1d88708f4 100644
--- a/ppapi/api/private/ppb_nacl_private.idl
+++ b/ppapi/api/private/ppb_nacl_private.idl
@@ -122,13 +122,15 @@ interface PPB_NaCl_Private {
                     [in] PP_CompletionCallback callback);
 
   /* Report to the browser that translation of the pexe for |instance|
-   * has finished. The browser may then store the translation in the
-   * cache. The renderer must first have called GetNexeFd for the same
-   * instance. (It is not guaranteed to, however; if there is an error
+   * has finished, or aborted with an error. If |success| is true, the
+   * browser may then store the translation in the cache. The renderer
+   * must first have called GetNexeFd for the same instance. (It is not
+   * guaranteed to, however; if there is an error on the browser side

[jvoung (off chromium), 2013/08/06 18:35:28]

I forget, what was this last part of the comment about?  It sounds like there is
an exception when the renderer does *not* call GetNexeFd.  Doesn't call
GetNexeFd, but the callback is called w/ an error code if the file is too big
for the cache, etc. ?

[dmichael (off chromium), 2013/08/06 18:53:58]

nit: probably should have a comma after "on the browser side"

[Derek Schuff, 2013/08/06 21:21:13]

Clarified.

[Derek Schuff, 2013/08/06 21:21:13]

Done.

    * or the file is too big for the cache, or the browser is in incognito
    * mode, no notification will be delivered to the plugin.)
    */
-  void ReportTranslationFinished([in] PP_Instance instance);
+  void ReportTranslationFinished([in] PP_Instance instance,
+                                 [in] PP_Bool success);
 
   /* Return true if we are off the record.
    */