Fixing crash that may occur after destroying a VideoSendStream.

Fixing crash that may occur after destroying a VideoSendStream.

It was possible that even after a VideoSendStream was destroyed,
it remained registered as a BitrateAllocator observer, causing a
segfault later.

Committed: https://crrev.com/62411a21c9461265cc0fa69e947afafc00f8c807
Cr-Commit-Position: refs/heads/master@{#12067}

[Taylor Brandstetter, 2016-03-18 18:36:10]

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

[Taylor Brandstetter, 2016-03-18 18:36:11]

PTAL.

[pbos-webrtc, 2016-03-18 23:12:03]

pbos@webrtc.org changed reviewers:
+ pbos@webrtc.org

[pbos-webrtc, 2016-03-18 23:12:03]

lgtm

[Taylor Brandstetter, 2016-03-18 23:31:29]

The CQ bit was checked by deadbeef@webrtc.org

[commit-bot: I haz the power, 2016-03-18 23:31:33]

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

[commit-bot: I haz the power, 2016-03-19 01:32:17]

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

[commit-bot: I haz the power, 2016-03-19 01:32:18]

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

[mflodman, 2016-03-20 07:13:17]

LGTM, thanks Taylor!

[perkj_webrtc, 2016-03-20 13:35:35]

The CQ bit was checked by perkj@webrtc.org

[commit-bot: I haz the power, 2016-03-20 13:35:46]

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

[commit-bot: I haz the power, 2016-03-20 14:49:18]

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

[commit-bot: I haz the power, 2016-03-20 14:49:19]

Try jobs failed on following builders:
  android_rel on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/android_rel/builds/11898)

[pbos-webrtc, 2016-03-20 20:22:31]

The CQ bit was checked by pbos@webrtc.org

[commit-bot: I haz the power, 2016-03-20 20:22:41]

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

[commit-bot: I haz the power, 2016-03-20 21:24:53]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-20 21:25:03]

Description was changed from

==========
Fixing crash that may occur after destroying a VideoSendStream.

It was possible that even after a VideoSendStream was destroyed,
it remained registered as a BitrateAllocator observer, causing a
segfault later.
==========

to

==========
Fixing crash that may occur after destroying a VideoSendStream.

It was possible that even after a VideoSendStream was destroyed,
it remained registered as a BitrateAllocator observer, causing a
segfault later.

Committed: https://crrev.com/62411a21c9461265cc0fa69e947afafc00f8c807
Cr-Commit-Position: refs/heads/master@{#12067}
==========

[commit-bot: I haz the power, 2016-03-20 21:25:04]

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