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

Unified Diff: remoting/host/win/launch_process_with_token.cc

Issue 10828181: [Chromoting] Moving common logic responsible for launching child processes to WorkerProcessLauncher… (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 4 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: remoting/host/win/launch_process_with_token.cc
diff --git a/remoting/host/win/launch_process_with_token.cc b/remoting/host/win/launch_process_with_token.cc
index 1837a0558b4a07fe7192ead436fb86498c2ade38..bc09cc8fb97ae908b4722ca451e7296e4cb7228b 100644
--- a/remoting/host/win/launch_process_with_token.cc
+++ b/remoting/host/win/launch_process_with_token.cc
@@ -343,8 +343,8 @@ bool CreateSessionToken(uint32 session_id, ScopedHandle* token_out) {
bool LaunchProcessWithToken(const FilePath& binary,
const std::wstring& command_line,
- HANDLE user_token,
- base::Process* process_out) {
+ const ScopedHandle& user_token,
+ ScopedHandle* process_out) {
std::wstring application_name = binary.value();
base::win::ScopedProcessInformation process_info;
@@ -401,7 +401,7 @@ bool LaunchProcessWithToken(const FilePath& binary,
}
CHECK(process_info.IsValid());
- process_out->set_handle(process_info.TakeProcessHandle());
+ process_out->Set(process_info.TakeProcessHandle());
return true;
}

Powered by Google App Engine
This is Rietveld 408576698