Add scoped StartupInfoEx to base.

base/win/scoped_startup_info_ex.cc

+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "base/win/scoped_startup_info_ex.h"
+
+#include "base/logging.h"
+#include "base/win/windows_version.h"
+
+namespace {
+
+typedef BOOL (WINAPI *InitializeProcThreadAttributeListFunction)(
+    LPPROC_THREAD_ATTRIBUTE_LIST attribute_list,
+    DWORD attribute_count,
+    DWORD flags,
+    PSIZE_T size);
+static InitializeProcThreadAttributeListFunction
+    s_InitializeProcThreadAttributeList;
+
+typedef BOOL (WINAPI *UpdateProcThreadAttributeFunction)(
+    LPPROC_THREAD_ATTRIBUTE_LIST attribute_list,
+    DWORD flags,
+    DWORD_PTR attribute,
+    PVOID value,
+    SIZE_T size,
+    PVOID previous_value,
+    PSIZE_T return_size);
+static UpdateProcThreadAttributeFunction s_UpdateProcThreadAttribute;
+
+typedef VOID (WINAPI *DeleteProcThreadAttributeListFunction)(
+    LPPROC_THREAD_ATTRIBUTE_LIST lpAttributeList);
+static DeleteProcThreadAttributeListFunction s_DeleteProcThreadAttributeList;

[brettw, 2012/07/10 05:41:24]

I've never seen "s_..." notation. I'd probably name this like a local var
(delete_proc_thread...) even though it's weird.

[jschuh, 2012/07/10 16:14:50]

Done.

+
+}  // namespace
+
+namespace base {
+namespace win {
+
+ScopedStartupInfoEx::ScopedStartupInfoEx() {
+  memset(&startup_info_, 0, sizeof(startup_info_));
+
+  // Pre Windows Vista doesn't support STARTUPINFOEX.
+  if (base::win::GetVersion() < base::win::VERSION_VISTA) {
+    startup_info_.StartupInfo.cb = sizeof(STARTUPINFO);
+    return;
+  }
+
+  startup_info_.StartupInfo.cb = sizeof(startup_info_);
+
+  // Load the attribute API functions.
+  if (!s_InitializeProcThreadAttributeList ||
+      !s_UpdateProcThreadAttribute ||
+      !s_DeleteProcThreadAttributeList) {
+    HMODULE module = ::GetModuleHandleW(L"kernel32.dll");
+    s_InitializeProcThreadAttributeList =
+        reinterpret_cast<InitializeProcThreadAttributeListFunction>(
+            ::GetProcAddress(module, "InitializeProcThreadAttributeList"));
+    s_UpdateProcThreadAttribute =
+        reinterpret_cast<UpdateProcThreadAttributeFunction>(
+            ::GetProcAddress(module, "UpdateProcThreadAttribute"));
+    s_DeleteProcThreadAttributeList =
+        reinterpret_cast<DeleteProcThreadAttributeListFunction>(
+            ::GetProcAddress(module, "DeleteProcThreadAttributeList"));
+  }
+}
+
+ScopedStartupInfoEx::~ScopedStartupInfoEx() {
+  if (startup_info_.lpAttributeList) {
+    s_DeleteProcThreadAttributeList(startup_info_.lpAttributeList);
+    delete [] reinterpret_cast<BYTE*>(startup_info_.lpAttributeList);
+  }
+}
+
+bool ScopedStartupInfoEx::InitializeProcThreadAttributeList(
+    DWORD attribute_count) {
+  if (startup_info_.StartupInfo.cb != sizeof(startup_info_) ||
+      startup_info_.lpAttributeList)
+    return false;
+
+  SIZE_T size = 0;
+  s_InitializeProcThreadAttributeList(NULL, attribute_count, 0, &size);
+  if (size == 0)
+    return false;
+
+  startup_info_.lpAttributeList =
+      reinterpret_cast<LPPROC_THREAD_ATTRIBUTE_LIST>(new BYTE[size]);
+  if (!s_InitializeProcThreadAttributeList(startup_info_.lpAttributeList,
+                                           attribute_count, 0, &size)) {
+    delete [] reinterpret_cast<BYTE*>(startup_info_.lpAttributeList);

[brettw, 2012/07/10 05:41:24]

I think you need to set the list to null after this.

[jschuh, 2012/07/10 16:14:50]

Done.

+    return false;
+  }
+
+  return true;
+}
+
+bool ScopedStartupInfoEx::UpdateProcThreadAttribute(
+    DWORD_PTR attribute,
+    void* value,
+    size_t size) {
+  if (!startup_info_.lpAttributeList)
+    return false;
+  return !!s_UpdateProcThreadAttribute(startup_info_.lpAttributeList, 0,
+                                       attribute, value, size, NULL, NULL);
+}
+
+}  // namespace win
+}  // namespace base
+