commit 7c4d20fd6c95f76cf909669b94effdbef05ecb54
author pbos@webrtc.org <pbos@webrtc.org> Thu Feb 12 12:20:08 2015
committer pbos@webrtc.org <pbos@webrtc.org> Thu Feb 12 12:20:50 2015
tree 11cbc49b129eb2ea765ae3dd89cbb2fe5e975cbf
parent ff689be3c0c59c1be29aaa0697aa0f762566d6c6

Remove potential deadlock in RTPSenderAudio.

Removes lock-order inversion formed by RTPSenderAudio->RTPSender calls
by doing a lot shorter locking which fetches a current state of
RTPSenderAudio variables before sending.

Thread annotates locked variables and removes one lock in
RTPSenderAudio, bonus fixes data races reported in voe_auto_test
--automated under TSan (DTMF data race).

Also includes some bonus cleanup of RTPSenderVideo which removes the
send critsect completely as all methods using it was always called
from RTPSender under its send_critsect.

R=henrik.lundin@webrtc.org, stefan@webrtc.org, tommi@webrtc.org
BUG=3001, chromium:454654

Review URL: https://webrtc-codereview.appspot.com/41869004

Cr-Commit-Position: refs/heads/master@{#8348}
git-svn-id: http://webrtc.googlecode.com/svn/trunk@8348 4adac7df-926f-26a2-2b94-8c16560cd09d