Add success status to ReportTranslationFinished in ppb_nacl_private and IPC

chrome/renderer/pepper/pnacl_translation_resource_host.h

Index: chrome/renderer/pepper/pnacl_translation_resource_host.h
diff --git a/chrome/renderer/pepper/pnacl_translation_resource_host.h b/chrome/renderer/pepper/pnacl_translation_resource_host.h
index 9f1a2040fc2ad00019d2a1887611694ec00fa92c..d542c373132e1a20927625ae076c3784d7c03d51 100644
--- a/chrome/renderer/pepper/pnacl_translation_resource_host.h
+++ b/chrome/renderer/pepper/pnacl_translation_resource_host.h
@@ -33,7 +33,7 @@ class PnaclTranslationResourceHost : public IPC::ChannelProxy::MessageFilter {
                      PP_Bool* is_hit,
                      PP_FileHandle* file_handle,
                      scoped_refptr<ppapi::TrackedCallback> callback);
-  void ReportTranslationFinished(PP_Instance instance);
+  void ReportTranslationFinished(PP_Instance instance, PP_Bool error);

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

replace "error" with "success" ?

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

Ah yeah... it was originally "error" but I replaced it with "success" to be
consistent with EnsurePnaclInstalledReply

 
   // Ensure that PNaCl resources (pnacl-llc.nexe, linker, libs) are installed.
   void EnsurePnaclInstalled(PP_Instance instance,
@@ -74,7 +74,8 @@ class PnaclTranslationResourceHost : public IPC::ChannelProxy::MessageFilter {
                          PP_Bool* is_hit,
                          PP_FileHandle* file_handle,
                          scoped_refptr<ppapi::TrackedCallback> callback);
-  void SendReportTranslationFinished(PP_Instance instance);
+  void SendReportTranslationFinished(PP_Instance instance,
+                                     PP_Bool success);
   void SendEnsurePnaclInstalled(PP_Instance instance,
                                 scoped_refptr<ppapi::TrackedCallback> callback);