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

Unified Diff: mojo/edk/test/multiprocess_test_helper.cc

Issue 2950153002: Improve process launch handle sharing API. (Closed)
Patch Set: Fix Created 3 years, 6 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: mojo/edk/test/multiprocess_test_helper.cc
diff --git a/mojo/edk/test/multiprocess_test_helper.cc b/mojo/edk/test/multiprocess_test_helper.cc
index ed96e1d0e98d1befca0a13f9f9bae9a1fed98ecb..3b5cfccc9e844e68fe6cf28a2b13ad79ab61e8eb 100644
--- a/mojo/edk/test/multiprocess_test_helper.cc
+++ b/mojo/edk/test/multiprocess_test_helper.cc
@@ -135,13 +135,10 @@ ScopedMessagePipeHandle MultiprocessTestHelper::StartChildWithExtraSwitch(
base::LaunchOptions options;
#if defined(OS_POSIX)
- options.fds_to_remap = &handle_passing_info;
+ options.fds_to_remap = handle_passing_info;
#elif defined(OS_WIN)
options.start_hidden = true;
- if (base::win::GetVersion() >= base::win::VERSION_VISTA)
- options.handles_to_inherit = &handle_passing_info;
- else
- options.inherit_handles = true;
+ options.inherit_mode = base::LaunchOptions::INHERIT_ALL;
#else
#error "Not supported yet."
#endif

Powered by Google App Engine
This is Rietveld 408576698