Write a log list for uploaded WebRTC logs.

chrome/browser/media/webrtc_log_uploader.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_UPLOADER_H_
 #define CHROME_BROWSER_MEDIA_WEBRTC_LOG_UPLOADER_H_
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
+#include "base/gtest_prod_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/platform_file.h"
 #include "net/url_request/url_fetcher_delegate.h"
 
 namespace base {
 class SharedMemory;
 }
 
 namespace net {
 class URLFetcher;
 class URLRequestContextGetter;
 }
 
 typedef struct z_stream_s z_stream;
 
 class WebRtcLogURLRequestContextGetter;
 
 // WebRtcLogUploader uploads WebRTC logs, keeps count of how many logs have
-// been started and denies further logs if a limit is reached. There must only
-// be one object of this type.
+// been started and denies further logs if a limit is reached. It also adds
+// the timestamp and report ID of the uploded log to a text file. There must
+// only be one object of this type.
 class WebRtcLogUploader : public net::URLFetcherDelegate {
  public:
-  WebRtcLogUploader();
+  // |upload_list_path| is the text file to add the uploaded log info to. It is
+  // intended for testing. If empty the default path will be used.
+  explicit WebRtcLogUploader(base::FilePath& upload_list_path);

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

instead of changing the constructor and add a FilePath dependency to the
production code, let's add a an inline method that tests can use to set the
path.  E.g.

void SetUploadPathForTesting(const base::FilePath& path) {
  upload_list_path_ = path;
}

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

Done.

   virtual ~WebRtcLogUploader();
 
   // net::URLFetcherDelegate implementation.
   virtual void OnURLFetchComplete(const net::URLFetcher* source) OVERRIDE;
   virtual void OnURLFetchUploadProgress(const net::URLFetcher* source,
                                         int64 current, int64 total) OVERRIDE;
 
   // Returns true is number of logs limit is not reached yet. Increases log
   // count if true is returned. Must be called before UploadLog().
   bool ApplyForStartLogging();
 
   // Uploads log and decreases log count. May only be called if permission to
   // to log has been granted by calling ApplyForStartLogging() and getting true
   // in return. After UploadLog has been called, a new permission must be
   // granted.
   void UploadLog(net::URLRequestContextGetter* request_context,
                  scoped_ptr<base::SharedMemory> shared_memory,
                  uint32 length,
                  const std::string& app_session_id,
                  const std::string& app_url);
 
  private:
+  FRIEND_TEST_ALL_PREFIXES(WebRtcLogUploaderTest, Basic);
+
   // Sets up a multipart body to be uploaded. The body is produced according
   // to RFC 2046.
   void SetupMultipart(std::string* post_data, uint8* log_buffer,
                       uint32 log_buffer_length,
                       const std::string& app_session_id,
                       const std::string& app_url);
 
   void AddLogData(std::string* post_data, uint8* log_buffer,
                   uint32 log_buffer_length);
   void CompressLog(std::string* post_data, uint8* input, uint32 input_size);
   void ResizeForNextOutput(std::string* post_data, z_stream* stream);
   void DecreaseLogCount();
 
+  // Append information (time and report ID) about this uploaded log to a log
+  // list file, limited to |kLogListLimitLines| entries. This list is used for
+  // viewing the uploaded logs under chrome://webrtc-logs, see
+  // WebRtcLogUploadList. The list has the format
+  // time,id
+  // time,id
+  // etc.
+  // where each line represents an uploaded log and "time" is Unix time.
+  void AddUploadedLogInfoToUploadListFile(const std::string& report_id);
+
   int log_count_;
+  base::FilePath upload_list_path_;
 
   DISALLOW_COPY_AND_ASSIGN(WebRtcLogUploader);
 };
 
 #endif  // CHROME_BROWSER_MEDIA_WEBRTC_LOG_UPLOADER_H_