Add success status to ReportTranslationFinished in ppb_nacl_private and IPC

chrome/browser/nacl_host/pnacl_host.cc

Index: chrome/browser/nacl_host/pnacl_host.cc
diff --git a/chrome/browser/nacl_host/pnacl_host.cc b/chrome/browser/nacl_host/pnacl_host.cc
index 6eb3ea0a9b77bacd0a75dfa850a00f4d8e513d87..9bb5b727e14ca23f225c0d10b9dd31e83d46bba5 100644
--- a/chrome/browser/nacl_host/pnacl_host.cc
+++ b/chrome/browser/nacl_host/pnacl_host.cc
@@ -180,7 +180,9 @@ void PnaclHost::GetNexeFd(int render_process_id,
 
 /////////////////// Cleanup
 
-void PnaclHost::TranslationFinished(int render_process_id, int pp_instance) {
+void PnaclHost::TranslationFinished(int render_process_id,
+                                    int pp_instance,
+                                    bool success) {

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

You're not actually doing anything with |success|. Is that intentional? It seems
like there ought to at least be a TODO, and IMHO it's helpful to comment out
success:
bool /* success */
to make it obvious it's unused.

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

I have another CL currently in flight which uses it. 

   DCHECK(thread_checker_.CalledOnValidThread());
   if (cache_state_ != CacheReady)
     return;