Pepper: SelLdrLauncherChrome cleanup.

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

Index: ppapi/native_client/src/trusted/plugin/sel_ldr_launcher_chrome.h
diff --git a/ppapi/native_client/src/trusted/plugin/sel_ldr_launcher_chrome.h b/ppapi/native_client/src/trusted/plugin/sel_ldr_launcher_chrome.h
index 4970b116c4d5848e7be25f81da128fbfc75f3779..4a5365e0f17148033e6a6e9f11dd9e9aec284a67 100644
--- a/ppapi/native_client/src/trusted/plugin/sel_ldr_launcher_chrome.h
+++ b/ppapi/native_client/src/trusted/plugin/sel_ldr_launcher_chrome.h
@@ -15,18 +15,10 @@ 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