diff options
| author | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-08-22 19:45:29 +0000 |
|---|---|---|
| committer | teravest@chromium.org <teravest@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-08-22 19:46:44 +0000 |
| commit | 9cfb9ff5222bb57d385bba5a600c038f08130cbe (patch) | |
| tree | cdab4ecae7f24d18dd6d60aa34f68170a164011b /ppapi/native_client/src/trusted/plugin/service_runtime.cc | |
| parent | 795075576d75fa10bfd3d4100b8224ff7962e7e2 (diff) | |
| download | chromium_src-9cfb9ff5222bb57d385bba5a600c038f08130cbe.zip chromium_src-9cfb9ff5222bb57d385bba5a600c038f08130cbe.tar.gz chromium_src-9cfb9ff5222bb57d385bba5a600c038f08130cbe.tar.bz2 | |
Pepper: Report NaCl exit status over Chromium IPC.
This change uses a NaCl embedder interface for getting the exit status inside
the loader process instead of receiving that information over SRPC.
The IPC message introduced here must be synchronous so that we're guaranteed to
report the exit status before the loader process exits.
BUG=397161
TEST=NaClBrowserTest.ExitStatus*
R=dmichael@chromium.org, jschuh@chromium.org, mseaborn@chromium.org
Review URL: https://codereview.chromium.org/484783002
Cr-Commit-Position: refs/heads/master@{#291480}
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@291480 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi/native_client/src/trusted/plugin/service_runtime.cc')
| -rw-r--r-- | ppapi/native_client/src/trusted/plugin/service_runtime.cc | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/ppapi/native_client/src/trusted/plugin/service_runtime.cc b/ppapi/native_client/src/trusted/plugin/service_runtime.cc index d570f4c..ffce563 100644 --- a/ppapi/native_client/src/trusted/plugin/service_runtime.cc +++ b/ppapi/native_client/src/trusted/plugin/service_runtime.cc @@ -238,7 +238,8 @@ void PluginReverseInterface::ReportCrash() { } void PluginReverseInterface::ReportExitStatus(int exit_status) { - service_runtime_->set_exit_status(exit_status); + // We do nothing here; reporting exit status is handled through a separate + // embedder interface. } int64_t PluginReverseInterface::RequestQuotaForWrite( @@ -582,10 +583,4 @@ ServiceRuntime::~ServiceRuntime() { NaClMutexDtor(&mu_); } -void ServiceRuntime::set_exit_status(int exit_status) { - nacl::MutexLocker take(&mu_); - if (main_service_runtime_) - plugin_->set_exit_status(exit_status & 0xff); -} - } // namespace plugin |