Convenience functions to convert RtcEvents to webrtc types.

webrtc/video/rtc_event_log.cc

Index: webrtc/video/rtc_event_log.cc
diff --git a/webrtc/video/rtc_event_log.cc b/webrtc/video/rtc_event_log.cc
index 476ee2afd79c82b9be6b41feb7424939842d942b..c9d12b4644d5203cc15bc47e3347adf379915cef 100644
--- a/webrtc/video/rtc_event_log.cc
+++ b/webrtc/video/rtc_event_log.cc
@@ -10,6 +10,7 @@
 
 #include "webrtc/video/rtc_event_log.h"
 
+#include <string>

[ivoc, 2015/08/25 15:25:55]

I don't think this include is needed here, as it is already included in
rtc_event_log.h. 

[hlundin-webrtc, 2015/08/28 10:37:16]

True. If it cannot be avoided in the .h file, it should not be included in the
.cc file.

[terelius, 2015/09/23 11:41:17]

Done.

[terelius, 2015/09/23 11:41:17]

Acknowledged.

 #include <deque>
 
 #include "webrtc/base/checks.h"
@@ -381,21 +382,6 @@ void RtcEventLogImpl::AddRecentEvent(const rtclog::Event& event) {
   }
 }
 
-bool RtcEventLog::ParseRtcEventLog(const std::string& file_name,
-                                   rtclog::EventStream* result) {
-  char tmp_buffer[1024];
-  int bytes_read = 0;
-  rtc::scoped_ptr<FileWrapper> dump_file(FileWrapper::Create());
-  if (dump_file->OpenFile(file_name.c_str(), true) != 0) {
-    return false;
-  }
-  std::string dump_buffer;
-  while ((bytes_read = dump_file->Read(tmp_buffer, sizeof(tmp_buffer))) > 0) {
-    dump_buffer.append(tmp_buffer, bytes_read);
-  }
-  dump_file->CloseFile();
-  return result->ParseFromString(dump_buffer);
-}
 
 #endif  // ENABLE_RTC_EVENT_LOG
 
@@ -403,4 +389,5 @@ bool RtcEventLog::ParseRtcEventLog(const std::string& file_name,
 rtc::scoped_ptr<RtcEventLog> RtcEventLog::Create() {
   return rtc::scoped_ptr<RtcEventLog>(new RtcEventLogImpl());
 }
+
 }  // namespace webrtc