NaCl plugin: Make ServiceRuntime::Shutdown() kill the NaCl loader

components/nacl/renderer/nexe_load_manager.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_NEXE_LOAD_MANAGER_H_
 #define COMPONENTS_NACL_RENDERER_NEXE_LOAD_MANAGER_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 50 matching lines @@
   // TODO(dmichael): Everything below this comment should eventually be made
   // private, when ppb_nacl_private_impl.cc is no longer using them directly.
   // The intent is for this class to only expose functions for reporting a
   // load state transition (e.g., ReportLoadError, ReportProgress,
   // ReportLoadAbort, etc.)
   void set_trusted_plugin_channel(
       std::unique_ptr<TrustedPluginChannel> channel);
   void set_manifest_service_channel(
       std::unique_ptr<ManifestServiceChannel> channel);
 
+  void close_trusted_plugin_channel();

[bbudge, 2016/12/08 23:20:00]

nit: not a getter or setter, so use camel case and move decl with other regular
methods.

[Derek Schuff, 2016/12/09 00:11:34]

Done.

+
   PP_NaClReadyState nacl_ready_state();
   void set_nacl_ready_state(PP_NaClReadyState ready_state);
 
   void SetReadOnlyProperty(PP_Var key, PP_Var value);
   void SetLastError(const std::string& error);
   void LogToConsole(const std::string& message);
 
   bool is_installed() const { return is_installed_; }
 
   int32_t exit_status() const { return exit_status_; }
@@ skipping 107 matching lines @@
   base::SharedMemoryHandle crash_info_shmem_handle_;
 
   std::unique_ptr<TrustedPluginChannel> trusted_plugin_channel_;
   std::unique_ptr<ManifestServiceChannel> manifest_service_channel_;
   base::WeakPtrFactory<NexeLoadManager> weak_factory_;
 };
 
 }  // namespace nacl
 
 #endif  // COMPONENTS_NACL_RENDERER_NEXE_LOAD_MANAGER_H_