From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Jeremy Rose <jeremya@chromium.org>
Date: Mon, 2 Aug 2021 15:56:56 -0700
Subject: only create a console if logging to stderr

This fixes an issue on Windows where, when `--enable-logging=file` is
passed and the app was not run from the console, a console window would
be created for each child process, despite logs being redirected to a
file.

diff --git a/content/app/content_main.cc b/content/app/content_main.cc
index bb2d3e991195877264fca46abd224d3a6ffd17e8..fc4f271f54235d2d8d823fdb0d95e3c76243390f 100644
--- a/content/app/content_main.cc
+++ b/content/app/content_main.cc
@@ -387,8 +387,12 @@ RunContentProcess(ContentMainParams params,
 
 #if BUILDFLAG(IS_WIN)
     // Route stdio to parent console (if any) or create one.
-    if (base::CommandLine::ForCurrentProcess()->HasSwitch(
-            switches::kEnableLogging)) {
+    auto* cmd_line = base::CommandLine::ForCurrentProcess();
+    bool log_to_stderr =
+        cmd_line->HasSwitch(switches::kEnableLogging) &&
+        cmd_line->GetSwitchValueASCII(switches::kEnableLogging) != "file" &&
+        !cmd_line->HasSwitch(switches::kLogFile);
+    if (log_to_stderr) {
       base::RouteStdioToConsole(true);
     }
 #endif
