Set up a NaCl load status callback to start replacing "start_module".

components/nacl/renderer/trusted_plugin_channel.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_NACL_RENDERER_TRUSTED_PLUGIN_CHANNEL_H_
 #define COMPONENTS_NACL_RENDERER_TRUSTED_PLUGIN_CHANNEL_H_
 
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
 #include "ipc/ipc_listener.h"
@@ skipping 10 matching lines @@
 }  // namespace IPC
 
 namespace nacl {
 class NexeLoadManager;
 
 class TrustedPluginChannel : public IPC::Listener {
  public:
   TrustedPluginChannel(NexeLoadManager* nexe_load_manager,
                        const IPC::ChannelHandle& handle,
                        base::WaitableEvent* shutdown_event,
-                       bool report_exit_status);
+                       bool main_service_runtime);

[Mark Seaborn, 2015/04/17 23:30:58]

This name is rather cryptic and its meaning isn't commented in the existing
code.  Maybe you could use a more descriptive name such as "is_helper_nexe" or
"is_pnacl_translator", and invert the variable?

[jvoung (off chromium), 2015/04/18 01:00:46]

Done.

   ~TrustedPluginChannel() override;
 
   bool Send(IPC::Message* message);
 
   // Listener implementation.
   bool OnMessageReceived(const IPC::Message& message) override;
   void OnChannelError() override;
 
   void OnReportExitStatus(int exit_status);
+  void OnReportLoadStatus(int load_status);
 
  private:
   // Non-owning pointer. This is safe because the TrustedPluginChannel is owned
   // by the NexeLoadManager pointed to here.
   NexeLoadManager* nexe_load_manager_;
   scoped_ptr<IPC::SyncChannel> channel_;
-  bool report_exit_status_;
+  bool main_service_runtime_;
 
   DISALLOW_COPY_AND_ASSIGN(TrustedPluginChannel);
 };
 
 }  // namespace nacl
 
 #endif  // COMPONENTS_NACL_RENDERER_TRUSTED_PLUGIN_CHANNEL_H_