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

Unified Diff: chrome/browser/nacl_host/nacl_process_host.h

Issue 2832093: NaCl: Allow setting up multiple sockets for subprocess instead of just one (Closed) Base URL: git://codf21.jail/chromium.git
Patch Set: Whitespace fixes Created 10 years, 5 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: chrome/browser/nacl_host/nacl_process_host.h
diff --git a/chrome/browser/nacl_host/nacl_process_host.h b/chrome/browser/nacl_host/nacl_process_host.h
index a5212de7b3df4267464c13834637624caad5667d..618492a1255929815c976ca115473bf32f71e0b7 100644
--- a/chrome/browser/nacl_host/nacl_process_host.h
+++ b/chrome/browser/nacl_host/nacl_process_host.h
@@ -29,7 +29,7 @@ class NaClProcessHost : public BrowserChildProcessHost {
// Initialize the new NaCl process, returning true on success.
bool Launch(ResourceMessageFilter* resource_message_filter,
- const int descriptor,
+ int socket_count,
IPC::Message* reply_msg);
virtual void OnMessageReceived(const IPC::Message& msg);
@@ -69,11 +69,9 @@ class NaClProcessHost : public BrowserChildProcessHost {
// The reply message to send.
IPC::Message* reply_msg_;
- // The socket pair for the NaCl process.
- nacl::Handle pair_[2];
-
- // The NaCl specific descriptor for this process.
- int descriptor_;
+ // Socket pairs for the NaCl process and renderer.
+ std::vector<nacl::Handle> sockets_for_renderer_;
+ std::vector<nacl::Handle> sockets_for_sel_ldr_;
gregoryd 2010/07/29 23:46:09 socket_for_loader would be better, Chrome people d
// Windows platform flag
bool running_on_wow64_;
« no previous file with comments | « no previous file | chrome/browser/nacl_host/nacl_process_host.cc » ('j') | chrome/common/nacl_messages_internal.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698