Make starting logging to a file be an explicit method on ChromeNetLog,
rather than be part of constructor.
Review-Url: https://codereview.chromium.org/2964263002
Cr-Commit-Position: refs/heads/master@{#484371}
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 5f9cd96..67dbf63 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -236,12 +236,14 @@
print_job_manager_.reset(new printing::PrintJobManager);
#endif
- base::FilePath net_log_path;
- if (command_line.HasSwitch(switches::kLogNetLog))
- net_log_path = command_line.GetSwitchValuePath(switches::kLogNetLog);
- net_log_.reset(new net_log::ChromeNetLog(
- net_log_path, GetNetCaptureModeFromCommandLine(command_line),
- command_line.GetCommandLineString(), chrome::GetChannelString()));
+ net_log_ = base::MakeUnique<net_log::ChromeNetLog>();
+
+ if (command_line.HasSwitch(switches::kLogNetLog)) {
+ net_log_->StartWritingToFile(
+ command_line.GetSwitchValuePath(switches::kLogNetLog),
+ GetNetCaptureModeFromCommandLine(command_line),
+ command_line.GetCommandLineString(), chrome::GetChannelString());
+ }
ChildProcessSecurityPolicy::GetInstance()->RegisterWebSafeScheme(
chrome::kChromeSearchScheme);