Investigate RelaunchChromeBrowserWithNewCommandLineIfNeeded log spam

chrome/app/chrome_exe_main_win.cc

 // Copyright (c) 2011 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 <windows.h>
 #include <malloc.h>
 #include <shellscalingapi.h>
 #include <stddef.h>
 #include <tchar.h>
 
@@ skipping 226 matching lines @@
 
   if (AttemptFastNotify(*command_line))
     return 0;
 
   RemoveAppCompatFlagsEntry();
 
   // Load and launch the chrome dll. *Everything* happens inside.
   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;
 }