Logging: pass logging related cmd line options to utility process

content/browser/utility_process_host_impl.cc

Index: content/browser/utility_process_host_impl.cc
diff --git a/content/browser/utility_process_host_impl.cc b/content/browser/utility_process_host_impl.cc
index 82c3c5330ebc931c9f1a4a1c9dc816d8bca9692d..30e54536db7fca7dc8eac186cbb6efefd1c11fbf 100644
--- a/content/browser/utility_process_host_impl.cc
+++ b/content/browser/utility_process_host_impl.cc
@@ -4,6 +4,7 @@
 
 #include "content/browser/utility_process_host_impl.h"
 
+#include "base/base_switches.h"
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/command_line.h"
@@ -165,6 +166,14 @@ bool UtilityProcessHostImpl::StartProcess() {
   cmd_line->AppendSwitchPath(switches::kUtilityProcessAllowedDir, exposed_dir_);
 #endif
 
+  static const char* kForwardSwitches[] = {
+    switches::kEnableLogging,  // Support, e.g., --enable-logging=stderr

[piman, 2012/10/12 17:00:33]

nit: I don't think this comment adds anything. Please remove?

+    switches::kV,
+    switches::kVModule,
+  };

[piman, 2012/10/12 17:00:33]

Could you move the flags handling above into here, when it's of the type if
(browser_command_line.HasSwitch(kFlag)) cmd_line->AppendSwitch(kFlag);
It's the same thing, but this way is more concise/readable.

+  cmd_line->CopySwitchesFrom(browser_command_line, kForwardSwitches,
+                             arraysize(kForwardSwitches));
+
   bool use_zygote = false;
 
 #if defined(OS_LINUX)