Fixing TSan data race warning in video end-to-end tests.

Fixing TSan data race warning in video end-to-end tests.

Needed to use critical section in "SendRtp"/"SendRtcp", which is what
the real implementation ultimately does.

TBR=stefan@webrtc.org

Committed: https://crrev.com/14cf12b1eaf6155dc474c68c5562597157c65b01
Cr-Commit-Position: refs/heads/master@{#13857}

[Taylor Brandstetter, 2016-08-22 22:44:16]

Description was changed from

==========
Fixing TSan data race warning in video end-to-end tests.

Needed to use critical section in "SendRtp"/"SendRtcp", which is what
the real implementation ultimately does.
==========

to

==========
Fixing TSan data race warning in video end-to-end tests.

Needed to use critical section in "SendRtp"/"SendRtcp", which is what
the real implementation ultimately does.

TBR=stefan@webrtc.org
==========

[Taylor Brandstetter, 2016-08-22 22:44:16]

deadbeef@webrtc.org changed reviewers:
+ stefan@webrtc.org

[Taylor Brandstetter, 2016-08-22 22:45:08]

PTAL. Here's an example of a test that failed for this reason:
https://build.chromium.org/p/client.webrtc/builders/Linux%20Tsan%20v2/builds/...

[Taylor Brandstetter, 2016-08-22 22:45:23]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2016-08-22 22:45:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2016-08-23 00:45:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-23 00:45:59]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[Taylor Brandstetter, 2016-08-23 01:14:25]

Description was changed from

==========
Fixing TSan data race warning in video end-to-end tests.

Needed to use critical section in "SendRtp"/"SendRtcp", which is what
the real implementation ultimately does.

TBR=stefan@webrtc.org
==========

to

==========
Fixing TSan data race warning in video end-to-end tests.

Needed to use critical section in "SendRtp"/"SendRtcp", which is what
the real implementation ultimately does.

TBR=stefan@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/14cf12b1eaf6155dc474c68c5...
==========

[Taylor Brandstetter, 2016-08-23 01:14:26]

Committed patchset #1 (id:1) manually as
14cf12b1eaf6155dc474c68c5562597157c65b01 (presubmit successful).

[commit-bot: I haz the power, 2016-08-23 01:14:26]

Description was changed from

==========
Fixing TSan data race warning in video end-to-end tests.

Needed to use critical section in "SendRtp"/"SendRtcp", which is what
the real implementation ultimately does.

TBR=stefan@webrtc.org

Committed:
https://chromium.googlesource.com/external/webrtc/+/14cf12b1eaf6155dc474c68c5...
==========

to

==========
Fixing TSan data race warning in video end-to-end tests.

Needed to use critical section in "SendRtp"/"SendRtcp", which is what
the real implementation ultimately does.

TBR=stefan@webrtc.org

Committed: https://crrev.com/14cf12b1eaf6155dc474c68c5562597157c65b01
Cr-Commit-Position: refs/heads/master@{#13857}
==========