Relocate Windows ProcessNeedsProfileDir to install_static

chrome/install_static/install_util.cc

Index: chrome/install_static/install_util.cc
diff --git a/chrome/install_static/install_util.cc b/chrome/install_static/install_util.cc
index 7a54c7196a1ea001a008cc90e31fa8fe3415c316..47a7597d25b62ce393a544ebc358586fee378bc4 100644
--- a/chrome/install_static/install_util.cc
+++ b/chrome/install_static/install_util.cc
@@ -64,6 +64,10 @@ constexpr wchar_t kRegValueName[] = L"name";
 constexpr wchar_t kRegValueUsageStats[] = L"usagestats";
 constexpr wchar_t kMetricsReportingEnabled[] = L"MetricsReportingEnabled";
 
+constexpr wchar_t kCloudPrintServiceProcess[] = L"cloud-print-service";
+constexpr wchar_t kNaClBrokerProcess[] = L"nacl-broker";

[grt (UTC plus 2), 2017/05/17 20:10:36]

#if !defined(DISABLE_NACL)

[manzagop (departed), 2017/05/17 21:15:56]

Done.

+constexpr wchar_t kNaClLoaderProcess[] = L"nacl-loader";
+
 void Trace(const wchar_t* format_string, ...) {
   static const int kMaxLogBufferSize = 1024;
   static wchar_t buffer[kMaxLogBufferSize] = {};
@@ -288,6 +292,42 @@ std::wstring ChannelFromAdditionalParameters(const InstallConstants& mode,
   return std::wstring();
 }
 
+// Converts a process type specified as a string to the ProcessType enum.
+ProcessType GetProcessType(std::wstring process_type) {

[grt (UTC plus 2), 2017/05/17 20:10:36]

const&

[manzagop (departed), 2017/05/17 21:15:56]

Err... Done.

+  if (process_type.empty()) {

[grt (UTC plus 2), 2017/05/17 20:10:36]

no braces and no "else":

  if (foo)
    return foo;
  if (bar)
    return bar;

[manzagop (departed), 2017/05/17 21:15:56]

Done.

+    return ProcessType::BROWSER_PROCESS;
+  } else if (process_type == kCloudPrintServiceProcess) {
+    return ProcessType::CLOUD_PRINT_SERVICE_PROCESS;
+#if !defined(DISABLE_NACL)
+  } else if (process_type == kNaClBrokerProcess) {
+    return ProcessType::NACL_BROKER_PROCESS;
+  } else if (process_type == kNaClLoaderProcess) {
+    return ProcessType::NACL_LOADER_PROCESS;
+#endif
+  } else {
+    return ProcessType::OTHER_PROCESS;
+  }
+}
+
+// Returns whether |process_type| needs the profile directory.
+bool ProcessNeedsProfileDir(ProcessType process_type) {
+  // On windows we don't want subprocesses other than the browser process and

[grt (UTC plus 2), 2017/05/17 20:10:36]

nit: Windows

[manzagop (departed), 2017/05/17 21:15:56]

Done.

+  // service processes to be able to use the profile directory because if it
+  // lies on a network share the sandbox will prevent us from accessing it.
+
+  if (process_type == ProcessType::BROWSER_PROCESS ||

[grt (UTC plus 2), 2017/05/17 20:10:36]

to protect against future types being added to the enum but not here, i would go
with:
  switch (process_type) {
    case foo:
    case bar:
#if !defined...
    case bletch:
#endif
      return true;
    case theother:
      return false;
  }
  assert(false);
  return false;

by listing every case explicitly and omitting a "default" clause, future
additions to the enum will throw a compile error until this is adjusted
accordingly.

[manzagop (departed), 2017/05/17 21:15:56]

Done. It's just clang that throws an error in that case, right?

+      process_type == ProcessType::CLOUD_PRINT_SERVICE_PROCESS) {
+    return true;
+  }
+#if !defined(DISABLE_NACL)
+  if (process_type == ProcessType::kNaClBrokerProcess ||
+      process_type == ProcessType::kNaClLoaderProcess) {
+    return true;
+  }
+#endif
+  return false;
+}
+
 }  // namespace
 
 bool IsSystemInstall() {
@@ -454,29 +494,18 @@ bool ReportingIsEnforcedByPolicy(bool* crash_reporting_enabled) {
 
 void InitializeProcessType() {
   assert(g_process_type == ProcessType::UNINITIALIZED);
-  typedef bool (*IsSandboxedProcessFunc)();
-  IsSandboxedProcessFunc is_sandboxed_process_func =
-      reinterpret_cast<IsSandboxedProcessFunc>(
-          ::GetProcAddress(::GetModuleHandle(nullptr), "IsSandboxedProcess"));
-  if (is_sandboxed_process_func && is_sandboxed_process_func()) {
-    g_process_type = ProcessType::NON_BROWSER_PROCESS;
-    return;
-  }
-
-  // TODO(robertshield): Drop the command line check when we drop support for
-  // enabling chrome_elf in unsandboxed processes.
-  const wchar_t* command_line = GetCommandLine();
-  if (command_line && ::wcsstr(command_line, L"--type")) {
-    g_process_type = ProcessType::NON_BROWSER_PROCESS;
-    return;
-  }
-
-  g_process_type = ProcessType::BROWSER_PROCESS;
+  std::wstring process_type =
+      GetSwitchValueFromCommandLine(::GetCommandLine(), kProcessType);
+  g_process_type = GetProcessType(process_type);
 }
 
 bool IsNonBrowserProcess() {
   assert(g_process_type != ProcessType::UNINITIALIZED);
-  return g_process_type == ProcessType::NON_BROWSER_PROCESS;
+  return g_process_type != ProcessType::BROWSER_PROCESS;
+}
+
+bool ProcessNeedsProfileDir(const std::string& process_type) {

[grt (UTC plus 2), 2017/05/17 20:10:36]

part of me wants this function to be an export from chrome_elf that is imported
by chrome{,_child}.dll so that it doesn't need to recompute the process type. i
can see that that's a bigger change, though, and isn't really a huge savings.

[manzagop (departed), 2017/05/17 21:15:56]

Acknowledged.

+  return ProcessNeedsProfileDir(GetProcessType(UTF8ToUTF16(process_type)));
 }
 
 std::wstring GetCrashDumpLocation() {