Write a log list for uploaded WebRTC logs.

chrome/browser/browser_process_impl.cc

 // Copyright (c) 2012 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 "chrome/browser/browser_process_impl.h"
 
 #include <algorithm>
 #include <map>
 #include <vector>
 
@@ skipping 618 matching lines @@
   return media_file_system_registry_.get();
 #endif
 }
 
 bool BrowserProcessImpl::created_local_state() const {
     return created_local_state_;
 }
 
 #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
 
 // static
 void BrowserProcessImpl::RegisterPrefs(PrefRegistrySimple* registry) {
   registry->RegisterBooleanPref(prefs::kDefaultBrowserSettingEnabled,
                                 false);
   // This policy needs to be defined before the net subsystem is initialized,
   // so we do it here.
@@ skipping 391 matching lines @@
 }
 
 void BrowserProcessImpl::OnAutoupdateTimer() {
   if (CanAutorestartForUpdate()) {
     DLOG(WARNING) << "Detected update.  Restarting browser.";
     RestartBackgroundInstance();
   }
 }
 
 #endif  // (defined(OS_WIN) || defined(OS_LINUX)) && !defined(OS_CHROMEOS)