Pepper: Report NaCl exit status over Chromium IPC.

components/nacl/renderer/trusted_plugin_channel.h

Index: components/nacl/renderer/trusted_plugin_channel.h
diff --git a/components/nacl/renderer/trusted_plugin_channel.h b/components/nacl/renderer/trusted_plugin_channel.h
index cba26d65475b107dccc5a30b77fceede50582ab0..81ec1bf0a76e7817a0109f04129f38916c70a7e9 100644
--- a/components/nacl/renderer/trusted_plugin_channel.h
+++ b/components/nacl/renderer/trusted_plugin_channel.h
@@ -21,11 +21,14 @@ class SyncChannel;
 }  // namespace IPC
 
 namespace nacl {
+class NexeLoadManager;
 
 class TrustedPluginChannel : public IPC::Listener {
  public:
-  TrustedPluginChannel(const IPC::ChannelHandle& handle,
-                       base::WaitableEvent* shutdown_event);
+  TrustedPluginChannel(NexeLoadManager* nexe_load_manager,
+                       const IPC::ChannelHandle& handle,
+                       base::WaitableEvent* shutdown_event,
+                       bool report_exit_status);
   virtual ~TrustedPluginChannel();
 
   bool Send(IPC::Message* message);
@@ -33,8 +36,13 @@ class TrustedPluginChannel : public IPC::Listener {
   // Listener implementation.
   virtual bool OnMessageReceived(const IPC::Message& message) OVERRIDE;
 
+  void OnReportExitStatus(int exit_status);
+
  private:
+  // Non-owning pointer.

[Mark Seaborn, 2014/08/22 17:36:55]

Can you comment here why the non-owning pointer is memory-safe?  What ensures
that the lifetime of TrustedPluginChannel is <= that of NexeLoadManager?

[dmichael (off chromium), 2014/08/22 17:39:00]

NexeLoadManager owns the TrustedPluginChannel. Though a comment wouldn't hurt, I
suppose.

[teravest, 2014/08/22 18:54:56]

I've added a comment.

+  NexeLoadManager* nexe_load_manager_;
   scoped_ptr<IPC::SyncChannel> channel_;
+  bool report_exit_status_;
 
   DISALLOW_COPY_AND_ASSIGN(TrustedPluginChannel);
 };