Write a log list for uploaded WebRTC logs.

chrome/browser/browser_process_impl.cc

Index: chrome/browser/browser_process_impl.cc
diff --git a/chrome/browser/browser_process_impl.cc b/chrome/browser/browser_process_impl.cc
index 6673807f1b4a8ba24337e87802d4c40513f2125a..bffe16e08514ccac9c91979193f3f739280831a5 100644
--- a/chrome/browser/browser_process_impl.cc
+++ b/chrome/browser/browser_process_impl.cc
@@ -636,8 +636,10 @@ bool BrowserProcessImpl::created_local_state() const {
 
 #if defined(ENABLE_WEBRTC)
 WebRtcLogUploader* BrowserProcessImpl::webrtc_log_uploader() {
-  if (!webrtc_log_uploader_.get())
-    webrtc_log_uploader_.reset(new WebRtcLogUploader());
+  if (!webrtc_log_uploader_.get()) {
+    base::FilePath empty_path;
+    webrtc_log_uploader_.reset(new WebRtcLogUploader(empty_path));

[jochen (gone - plz use gerrit), 2013/07/01 11:46:26]

nit. Doesn't WebRtcLogUploader(base::FilePath()) work?

[tommi (sloooow) - chröme, 2013/07/01 12:01:01]

instead of requiring an empty path can we have two constructors, one default one
and separate explicit one that accepts a non-empty path?

[Henrik Grunell, 2013/07/01 13:26:22]

As per the next comment, I've removed the argument and added a function to set
for testing.

+  }
   return webrtc_log_uploader_.get();
 }
 #endif