Added log messages for important call setup events: first packet sent/received

webrtc/modules/rtp_rtcp/source/rtp_sender_audio.cc

Index: webrtc/modules/rtp_rtcp/source/rtp_sender_audio.cc
diff --git a/webrtc/modules/rtp_rtcp/source/rtp_sender_audio.cc b/webrtc/modules/rtp_rtcp/source/rtp_sender_audio.cc
index 804294ac5408547e9e347bd5a743348e9f26dc67..dfa010eba161cf48d9c318da9b72d13c71cd970f 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_sender_audio.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_sender_audio.cc
@@ -12,6 +12,7 @@
 
 #include <string.h>
 
+#include "webrtc/base/logging.h"
 #include "webrtc/base/trace_event.h"
 #include "webrtc/modules/rtp_rtcp/include/rtp_rtcp_defines.h"
 #include "webrtc/modules/rtp_rtcp/source/byte_io.h"
@@ -333,6 +334,7 @@ int32_t RTPSenderAudio::SendAudio(FrameType frameType,
       memcpy(dataBuffer + rtpHeaderLength, payloadData, payloadSize);
     }
   }
+
   {
     CriticalSectionScoped cs(_sendAudioCritsect.get());
     _lastPayloadType = payloadType;
@@ -348,10 +350,14 @@ int32_t RTPSenderAudio::SendAudio(FrameType frameType,
   TRACE_EVENT_ASYNC_END2("webrtc", "Audio", captureTimeStamp, "timestamp",
                          _rtpSender->Timestamp(), "seqnum",
                          _rtpSender->SequenceNumber());
-  return _rtpSender->SendToNetwork(dataBuffer, payloadSize, rtpHeaderLength,
-                                   TickTime::MillisecondTimestamp(),
-                                   kAllowRetransmission,
-                                   RtpPacketSender::kHighPriority);
+  int32_t sendResult = _rtpSender->SendToNetwork(

[stefan-webrtc, 2016/04/07 08:57:36]

send_result

[skvlad, 2016/04/07 21:36:34]

Done. The rest of the function seems to use camelCased variable names though.

+      dataBuffer, payloadSize, rtpHeaderLength,
+      TickTime::MillisecondTimestamp(), kAllowRetransmission,
+      RtpPacketSender::kHighPriority);
+  if (first_packet_sent_()) {
+    LOG(LS_INFO) << "First audio RTP packet sent";

[stefan-webrtc, 2016/04/07 08:57:36]

"sent to pacer" is a better description.

[skvlad, 2016/04/07 21:36:34]

Done.

+  }
+  return sendResult;
 }
 
 // Audio level magnitude and voice activity flag are set for each RTP packet