Investigate RelaunchChromeBrowserWithNewCommandLineIfNeeded log spam

chrome/app/chrome_exe_main_win.cc

Index: chrome/app/chrome_exe_main_win.cc
diff --git a/chrome/app/chrome_exe_main_win.cc b/chrome/app/chrome_exe_main_win.cc
index 22ec4bf4f40a6c8b4a9efa2e68ec11aef881fca3..8af352d5ca1f05f9dd2ff0578c7b92c2d659327c 100644
--- a/chrome/app/chrome_exe_main_win.cc
+++ b/chrome/app/chrome_exe_main_win.cc
@@ -244,7 +244,10 @@ int main() {
   VLOG(1) << "About to load main DLL.";
   MainDllLoader* loader = MakeMainDllLoader();
   int rc = loader->Launch(instance);
-  loader->RelaunchChromeBrowserWithNewCommandLineIfNeeded();
+  if (process_type.empty()) {

[grt (UTC plus 2), 2016/09/19 19:59:42]

it feels cleaner to me for this function to know as little as possible about how
different process types operate. wdyt about putting this logic in MainDllLoader?

[manzagop (departed), 2016/09/20 19:39:39]

Sgtm.

+    // Only for the browser process.
+    loader->RelaunchChromeBrowserWithNewCommandLineIfNeeded();
+  }
   delete loader;
   return rc;
 }