Pepper: SelLdrLauncherChrome cleanup.

ppapi/native_client/src/trusted/plugin/sel_ldr_launcher_chrome.h

 // Copyright (c) 2012 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 PPAPI_NATIVE_CLIENT_SRC_TRUSTED_PLUGIN_SEL_LDR_LAUNCHER_CHROME_H_
 #define PPAPI_NATIVE_CLIENT_SRC_TRUSTED_PLUGIN_SEL_LDR_LAUNCHER_CHROME_H_
 
 #include "native_client/src/trusted/nonnacl_util/sel_ldr_launcher.h"
 #include "ppapi/c/pp_instance.h"
 #include "ppapi/c/private/ppb_nacl_private.h"
 #include "ppapi/cpp/completion_callback.h"
 
 namespace plugin {
 
 class SelLdrLauncherChrome : public nacl::SelLdrLauncherBase {
  public:
   virtual bool Start(const char* url);
-  virtual void Start(PP_Instance instance,
-                     bool main_service_runtime,
-                     const char* url,
-                     const PP_NaClFileInfo* file_info,
-                     bool uses_irt,
-                     bool uses_ppapi,
-                     bool uses_nonsfi_mode,
-                     bool enable_ppapi_dev,
-                     bool enable_dyncode_syscalls,
-                     bool enable_exception_handling,
-                     bool enable_crash_throttling,
-                     pp::CompletionCallback callback);
+
+  // Provides a way for LaunchSelLdr() to write imc_channel information into
+  // this class.
+  NaClHandle* mutable_channel() { return &channel_; }

[bbudge, 2014/06/30 18:50:38]

It's strange to expose this with a getter here, although another subclass also
does this:
https://code.google.com/p/chromium/codesearch#chromium/src/native_client/src/...

Why not expose it on the base class?

 };
 
 }  // namespace plugin
 
 #endif