Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(514)

Unified Diff: components/nacl/renderer/trusted_plugin_channel.h

Issue 1090233003: Set up a NaCl load status callback to start replacing "start_module". (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: use the enum instead of int Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/nacl/renderer/trusted_plugin_channel.h
diff --git a/components/nacl/renderer/trusted_plugin_channel.h b/components/nacl/renderer/trusted_plugin_channel.h
index efeb5e8fe988834df922756ebb6924feef60d6ad..d657d5338e187d95de34eac1af8275abfd21f0d9 100644
--- a/components/nacl/renderer/trusted_plugin_channel.h
+++ b/components/nacl/renderer/trusted_plugin_channel.h
@@ -8,6 +8,7 @@
#include "base/callback.h"
#include "base/memory/scoped_ptr.h"
#include "ipc/ipc_listener.h"
+#include "native_client/src/trusted/service_runtime/nacl_error_code.h"
#include "ppapi/c/pp_instance.h"
namespace base {
@@ -28,7 +29,7 @@ class TrustedPluginChannel : public IPC::Listener {
TrustedPluginChannel(NexeLoadManager* nexe_load_manager,
const IPC::ChannelHandle& handle,
base::WaitableEvent* shutdown_event,
- bool report_exit_status);
+ bool is_helper_nexe);
~TrustedPluginChannel() override;
bool Send(IPC::Message* message);
@@ -38,13 +39,14 @@ class TrustedPluginChannel : public IPC::Listener {
void OnChannelError() override;
void OnReportExitStatus(int exit_status);
+ void OnReportLoadStatus(NaClErrorCode 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 is_helper_nexe_;
DISALLOW_COPY_AND_ASSIGN(TrustedPluginChannel);
};
« no previous file with comments | « components/nacl/renderer/ppb_nacl_private_impl.cc ('k') | components/nacl/renderer/trusted_plugin_channel.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698