Fixing a segfault that can occur when changing the track of an RtpSender.

Fixing a segfault that can occur when changing the track of an RtpSender.

The reference to the old track needs to be kept alive until SetAudioSend/
SetSource is called, because otherwise it could be deleted while the audio/
video engine is still trying to use the track.

BUG=webrtc:5796
Committed: https://crrev.com/5dd42fd849062373cab757605824254fc16db6b6
Cr-Commit-Position: refs/heads/master@{#12598}

[Taylor Brandstetter, 2016-04-18 22:06:53]

Description was changed from

==========
Fixing a segfault that can occur when changing the track of an RtpSender.

The reference to the old track needs to be kept alive until SetAudioSend/
SetSource is called, because otherwise it could be deleted while the audio/
video engine is still trying to use the track.

BUG=webrtc:5796
==========

to

==========
Fixing a segfault that can occur when changing the track of an RtpSender.

The reference to the old track needs to be kept alive until SetAudioSend/
SetSource is called, because otherwise it could be deleted while the audio/
video engine is still trying to use the track.

BUG=webrtc:5796
==========

[Taylor Brandstetter, 2016-04-18 22:06:54]

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

[pthatcher1, 2016-04-29 22:32:58]

lgtm

[Taylor Brandstetter, 2016-05-02 19:58:18]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2016-05-02 19:58:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1894283002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1894283002/1

[commit-bot: I haz the power, 2016-05-02 21:58:48]

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

[commit-bot: I haz the power, 2016-05-02 21:58:49]

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

[Taylor Brandstetter, 2016-05-02 23:18:16]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2016-05-02 23:18:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1894283002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1894283002/1

[commit-bot: I haz the power, 2016-05-02 23:20:05]

Description was changed from

==========
Fixing a segfault that can occur when changing the track of an RtpSender.

The reference to the old track needs to be kept alive until SetAudioSend/
SetSource is called, because otherwise it could be deleted while the audio/
video engine is still trying to use the track.

BUG=webrtc:5796
==========

to

==========
Fixing a segfault that can occur when changing the track of an RtpSender.

The reference to the old track needs to be kept alive until SetAudioSend/
SetSource is called, because otherwise it could be deleted while the audio/
video engine is still trying to use the track.

BUG=webrtc:5796
==========

[commit-bot: I haz the power, 2016-05-02 23:20:06]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-05-02 23:20:14]

Description was changed from

==========
Fixing a segfault that can occur when changing the track of an RtpSender.

The reference to the old track needs to be kept alive until SetAudioSend/
SetSource is called, because otherwise it could be deleted while the audio/
video engine is still trying to use the track.

BUG=webrtc:5796
==========

to

==========
Fixing a segfault that can occur when changing the track of an RtpSender.

The reference to the old track needs to be kept alive until SetAudioSend/
SetSource is called, because otherwise it could be deleted while the audio/
video engine is still trying to use the track.

BUG=webrtc:5796

Committed: https://crrev.com/5dd42fd849062373cab757605824254fc16db6b6
Cr-Commit-Position: refs/heads/master@{#12598}
==========

[commit-bot: I haz the power, 2016-05-02 23:20:15]

Patchset 1 (id:??) landed as
https://crrev.com/5dd42fd849062373cab757605824254fc16db6b6
Cr-Commit-Position: refs/heads/master@{#12598}