Pepper: Report NaCl exit status over Chromium IPC.

ppapi/native_client/src/trusted/plugin/service_runtime.cc

Index: ppapi/native_client/src/trusted/plugin/service_runtime.cc
diff --git a/ppapi/native_client/src/trusted/plugin/service_runtime.cc b/ppapi/native_client/src/trusted/plugin/service_runtime.cc
index d570f4cfc5b25f158c6682b9a59ecce4d77a3af4..ffce56320dedd85df0dc9ae4d7f82fa6291d2142 100644
--- a/ppapi/native_client/src/trusted/plugin/service_runtime.cc
+++ b/ppapi/native_client/src/trusted/plugin/service_runtime.cc
@@ -238,7 +238,8 @@ void PluginReverseInterface::ReportCrash() {
 }
 
 void PluginReverseInterface::ReportExitStatus(int exit_status) {

[dmichael (off chromium), 2014/08/22 16:41:07]

So can we get rid of this function?

[teravest, 2014/08/22 16:51:58]

Not yet.
nacl::ReverseInterface::ReportExitStatus is pure virtual.

After this change, there could be another NaCl change to remove the function
there, and then this one could be removed.

-  service_runtime_->set_exit_status(exit_status);
+  // We do nothing here; reporting exit status is handled through a separate
+  // embedder interface.
 }
 
 int64_t PluginReverseInterface::RequestQuotaForWrite(
@@ -582,10 +583,4 @@ ServiceRuntime::~ServiceRuntime() {
   NaClMutexDtor(&mu_);
 }
 
-void ServiceRuntime::set_exit_status(int exit_status) {
-  nacl::MutexLocker take(&mu_);
-  if (main_service_runtime_)
-    plugin_->set_exit_status(exit_status & 0xff);
-}
-
 }  // namespace plugin