Adding WebRTC log upload list under chrome://webrtc-logs

chrome/browser/media/webrtc_log_upload_list.h

+// Copyright 2013 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.
+
+#ifndef CHROME_BROWSER_MEDIA_WEBRTC_LOG_UPLOAD_LIST_H_
+#define CHROME_BROWSER_MEDIA_WEBRTC_LOG_UPLOAD_LIST_H_
+
+#include "chrome/browser/upload_list.h"
+
+// Loads and parses a text file list of uploaded WebRTC logs.
+class WebRtcLogUploadList : public UploadList {
+ public:
+  // Creates the WebRTC log upload list with the given callback delegate.
+  static WebRtcLogUploadList* Create(Delegate* delegate);
+
+  // Used in this class when reading the list file and in WebRtcLogUploader when
+  // writing to the list file.
+  static const char* kWebRtcLogListFilename;

[James Hawkins, 2013/06/24 18:31:06]

I still don't understand why this member var is public.  Is it WebRtcLogUploader
that needs it outside of this class?  This seems smelly, but my nose is stuffed.

[Henrik Grunell, 2013/06/24 20:29:34]

Yes that's correct. I did it the same way as for chrome://crashes, see
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/crash_upload_l...

+
+  // Creates a new WebRTC log upload list with the given callback delegate.
+  // |upload_log_path| is the full path to the file to read the list from.
+  explicit WebRtcLogUploadList(Delegate* delegate,
+                               const base::FilePath& upload_log_path);
+
+ protected:
+  virtual ~WebRtcLogUploadList();
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(WebRtcLogUploadList);
+};
+
+#endif  // CHROME_BROWSER_MEDIA_WEBRTC_LOG_UPLOAD_LIST_H_