Update to the RtcEventLog protobuf to remove the DebugEvent message.

webrtc/video/rtc_event_log.proto

 syntax = "proto2";
 option optimize_for = LITE_RUNTIME;
 package webrtc.rtclog;
 
 
 enum MediaType {
   ANY = 0;
   AUDIO = 1;
   VIDEO = 2;
   DATA = 3;
@@ skipping 11 matching lines @@
 
 message Event {
   // required - Elapsed wallclock time in us since the start of the log.
   optional int64 timestamp_us = 1;
 
   // The different types of events that can occur, the UNKNOWN_EVENT entry
   // is added in case future EventTypes are added, in that case old code will
   // receive the new events as UNKNOWN_EVENT.
   enum EventType {
     UNKNOWN_EVENT = 0;
-    RTP_EVENT = 1;
-    RTCP_EVENT = 2;
-    DEBUG_EVENT = 3;
-    VIDEO_RECEIVER_CONFIG_EVENT = 4;
-    VIDEO_SENDER_CONFIG_EVENT = 5;
-    AUDIO_RECEIVER_CONFIG_EVENT = 6;
-    AUDIO_SENDER_CONFIG_EVENT = 7;
+    LOG_START = 1;
+    LOG_END = 2;
+    RTP_EVENT = 3;
+    RTCP_EVENT = 4;
+    AUDIO_PLAYOUT_EVENT = 5;
+    VIDEO_RECEIVER_CONFIG_EVENT = 6;
+    VIDEO_SENDER_CONFIG_EVENT = 7;
+    AUDIO_RECEIVER_CONFIG_EVENT = 8;
+    AUDIO_SENDER_CONFIG_EVENT = 9;
   }
 
   // required - Indicates the type of this event
   optional EventType type = 2;
 
   // optional - but required if type == RTP_EVENT
   optional RtpPacket rtp_packet = 3;
 
   // optional - but required if type == RTCP_EVENT
   optional RtcpPacket rtcp_packet = 4;
 
-  // optional - but required if type == DEBUG_EVENT
-  optional DebugEvent debug_event = 5;
+  // optional - but required if type == AUDIO_PLAYOUT_EVENT
+  optional AudioPlayoutEvent audio_playout_event = 5;
 
   // optional - but required if type == VIDEO_RECEIVER_CONFIG_EVENT
   optional VideoReceiveConfig video_receiver_config = 6;
 
   // optional - but required if type == VIDEO_SENDER_CONFIG_EVENT
   optional VideoSendConfig video_sender_config = 7;
 
   // optional - but required if type == AUDIO_RECEIVER_CONFIG_EVENT
   optional AudioReceiveConfig audio_receiver_config = 8;
 
@@ skipping 23 matching lines @@
   // required - True if the packet is incoming w.r.t. the user logging the data
   optional bool incoming = 1;
 
   // required
   optional MediaType type = 2;
 
   // required - The whole packet including both payload and header.
   optional bytes packet_data = 3;
 }
 
-
-message DebugEvent {
-  // Indicates the type of the debug event.
-  // LOG_START and LOG_END indicate the start and end of the log respectively.
-  // AUDIO_PLAYOUT indicates a call to the PlayoutData10Ms() function in ACM.
-  enum EventType {
-    UNKNOWN_EVENT = 0;
-    LOG_START = 1;
-    LOG_END = 2;
-    AUDIO_PLAYOUT = 3;
-  }
-
-  // required
-  optional EventType type = 1;
-
-  // required if type == AUDIO_PLAYOUT
+message AudioPlayoutEvent {
+  // required - The SSRC of the audio stream associated with the playout event.
   optional uint32 local_ssrc = 2;
 }
 
 
 // TODO(terelius): Video and audio streams could in principle share SSRC,
 // so identifying a stream based only on SSRC might not work.
 // It might be better to use a combination of SSRC and media type
 // or SSRC and port number, but for now we will rely on SSRC only.
 message VideoReceiveConfig {
   // required - Synchronization source (stream identifier) to be received.
@@ skipping 94 matching lines @@
 message EncoderConfig {
   // required
   optional string name = 1;
 
   // required
   optional sint32 payload_type = 2;
 }
 
 
 message AudioReceiveConfig {
-  // TODO(terelius): Add audio-receive config.
+  // required - Synchronization source (stream identifier) to be received.
+  optional uint32 remote_ssrc = 1;
+
+  // required - Sender SSRC used for sending RTCP (such as receiver reports).
+  optional uint32 local_ssrc = 2;
+
+  // RTP header extensions used for the received audio stream.
+  repeated RtpHeaderExtension header_extensions = 3;
 }
 
 
 message AudioSendConfig {
-  // TODO(terelius): Add audio-receive config.
+  // required - Synchronization source (stream identifier) for outgoing stream.
+  optional uint32 ssrc = 1;

[terelius, 2015/10/02 09:42:02]

Could you check if this should be repeated instead of optional? At least for for
video, there is a vector of ssrcs.

[ivoc, 2015/10/05 14:33:14]

Yes, I checked this with several team members, and it seems like a single SSRC
is sufficient for the audio stream.

+
+  // RTP header extensions used for the outgoing audio stream.
+  repeated RtpHeaderExtension header_extensions = 2;
 }