remoting/host/win/launch_process_with_token.h - Issue 10828160: [Chromoting] Move CreateSessionToken() next to launch process utilities.

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

Issue 10828160: [Chromoting] Move CreateSessionToken() next to launch process utilities. (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

« no previous file with comments | « remoting/host/daemon_process_win.cc ('k') | remoting/host/win/launch_process_with_token.cc » ('j') | remoting/host/win/launch_process_with_token.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: remoting/host/win/launch_process_with_token.h

diff --git a/remoting/host/win/launch_process_with_token.h b/remoting/host/win/launch_process_with_token.h

index 8234aae0163e5c9648040edd0d0a57ec53f67e4f..54479d5a56391460453cb655deacdac15a10f417 100644

--- a/remoting/host/win/launch_process_with_token.h

+++ b/remoting/host/win/launch_process_with_token.h

@@ -10,9 +10,14 @@

#include "base/file_path.h"

#include "base/process_util.h"

+#include "base/win/scoped_handle.h"

Wez 2012/08/03 20:56:06 nit: Forward-declare since you don't actually need

alexeypa (please no reviews) 2012/08/03 21:40:39 ScopedHandle is a template specialization which ca

Wez 2012/08/06 17:41:04 Personally I'd leave the API using ScopedHandle, a

alexeypa (please no reviews) 2012/08/06 17:48:36 Getting rid of include wasn't the reason to switch

Wez 2012/08/06 17:57:30 Right; Pass() is syntactic sugar for Set(Take());

namespace remoting {

+// Creates a copy of the current process token for the given |session_id| so

+// it can be used to launch a process in that session.

+bool CreateSessionToken(uint32 session_id, base::win::ScopedHandle* token_out);

// Launches |binary| in the security context of the user represented by

// |user_token|. The session ID specified by the token is respected as well.

bool LaunchProcessWithToken(const FilePath& binary,

« no previous file with comments | « remoting/host/daemon_process_win.cc ('k') | remoting/host/win/launch_process_with_token.cc » ('j') | remoting/host/win/launch_process_with_token.cc » ('J')