commit 545727ecce444320328b825d65b287e844dca7cb
author pbos <pbos@webrtc.org> Wed Jul 01 13:31:06 2015
committer Commit bot <commit-bot@chromium.org> Wed Jul 01 13:31:14 2015
tree a58b52da68d4b3c1ffe3dc9b007661723070242d
parent bd2522abf75891f34da6f83c247c47ca95641cee

Move early-return in TimeToSendPadding.

Prevents taking send_critsect_ for checking sending status when not
actually intending to send padding.

BUG=
R=stefan@webrtc.org

Review URL: https://codereview.webrtc.org/1218093002

Cr-Commit-Position: refs/heads/master@{#9526}

webrtc/modules/rtp_rtcp/source/rtp_sender.cc

diff --git a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
index 8923135..29b2897 100644
--- a/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
+++ b/webrtc/modules/rtp_rtcp/source/rtp_sender.cc
@@ -958,12 +958,12 @@
 }
 
 size_t RTPSender::TimeToSendPadding(size_t bytes) {
+  if (bytes == 0)
+    return 0;
   {
     CriticalSectionScoped cs(send_critsect_.get());
     if (!sending_media_) return 0;
   }
-  if (bytes == 0)
-    return 0;
   size_t bytes_sent = TrySendRedundantPayloads(bytes);
   if (bytes_sent < bytes)
     bytes_sent += TrySendPadData(bytes - bytes_sent);