Log Call {audio, video} stream deletions.

Log Call {audio, video} stream deletions.

BUG=
R=solenberg@webrtc.org, stefan@webrtc.org

Committed: https://crrev.com/a2f30deea342896ee40cc4d90567f091efbe0fc9
Cr-Commit-Position: refs/heads/master@{#10286}

[pbos-webrtc, 2015-10-14 10:01:41]

PTAL, this should make logs a bit easier to understand

[pbos-webrtc, 2015-10-14 10:01:51]

The CQ bit was checked by pbos@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2015-10-14 10:02:15]

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

[pbos-webrtc, 2015-10-14 10:24:50]

move logging to ctor/dtor

[pbos-webrtc, 2015-10-14 10:25:36]

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

[pbos-webrtc, 2015-10-14 10:25:37]

Moved logging to ctor/dtor, ptal

+solenberg@ for webrtc/audio

[pbos-webrtc, 2015-10-14 10:29:01]

compile

[stefan-webrtc, 2015-10-14 10:30:16]

https://codereview.webrtc.org/1400333002/diff/20001/webrtc/call/call.cc
File webrtc/call/call.cc (right):

https://codereview.webrtc.org/1400333002/diff/20001/webrtc/call/call.cc#newco...
webrtc/call/call.cc:180: // logging to match others.
Rewrite these comments.

https://codereview.webrtc.org/1400333002/diff/20001/webrtc/video/video_send_s...
File webrtc/video/video_send_stream.cc (right):

https://codereview.webrtc.org/1400333002/diff/20001/webrtc/video/video_send_s...
webrtc/video/video_send_stream.cc:462: const VideoSendStream::Config&
VideoSendStream::config() const {
Not needed, right?

[the sun, 2015-10-14 13:25:29]

On 2015/10/14 10:30:16, stefan-webrtc (holmer) wrote:
> https://codereview.webrtc.org/1400333002/diff/20001/webrtc/call/call.cc
> File webrtc/call/call.cc (right):
> 
>
https://codereview.webrtc.org/1400333002/diff/20001/webrtc/call/call.cc#newco...
> webrtc/call/call.cc:180: // logging to match others.
> Rewrite these comments.
> 
>
https://codereview.webrtc.org/1400333002/diff/20001/webrtc/video/video_send_s...
> File webrtc/video/video_send_stream.cc (right):
> 
>
https://codereview.webrtc.org/1400333002/diff/20001/webrtc/video/video_send_s...
> webrtc/video/video_send_stream.cc:462: const VideoSendStream::Config&
> VideoSendStream::config() const {
> Not needed, right?

lgtm for webrtc/audio

[pbos-webrtc, 2015-10-15 10:21:39]

updated comment

[pbos-webrtc, 2015-10-15 10:21:43]

PTAL

https://codereview.webrtc.org/1400333002/diff/20001/webrtc/call/call.cc
File webrtc/call/call.cc (right):

https://codereview.webrtc.org/1400333002/diff/20001/webrtc/call/call.cc#newco...
webrtc/call/call.cc:180: // logging to match others.
On 2015/10/14 10:30:16, stefan-webrtc (holmer) wrote:
> Rewrite these comments.

Done.

https://codereview.webrtc.org/1400333002/diff/20001/webrtc/video/video_send_s...
File webrtc/video/video_send_stream.cc (right):

https://codereview.webrtc.org/1400333002/diff/20001/webrtc/video/video_send_s...
webrtc/video/video_send_stream.cc:462: const VideoSendStream::Config&
VideoSendStream::config() const {
On 2015/10/14 10:30:16, stefan-webrtc (holmer) wrote:
> Not needed, right?

Done.

[pbos-webrtc, 2015-10-15 11:36:16]

The CQ bit was checked by pbos@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2015-10-15 11:36:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1400333002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1400333002/60001

[stefan-webrtc, 2015-10-15 11:38:36]

lgtm

[pbos-webrtc, 2015-10-15 11:39:56]

The CQ bit was unchecked by pbos@webrtc.org

[pbos-webrtc, 2015-10-15 11:40:03]

The CQ bit was checked by pbos@webrtc.org

[pbos-webrtc, 2015-10-15 11:40:03]

The patchset sent to the CQ was uploaded after l-g-t-m from solenberg@webrtc.org
Link to the patchset: https://codereview.webrtc.org/1400333002/#ps60001 (title:
"updated comment")

[commit-bot: I haz the power, 2015-10-15 11:40:10]

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

[commit-bot: I haz the power, 2015-10-15 12:22:16]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-15 12:22:30]

Patchset 4 (id:??) landed as
https://crrev.com/a2f30deea342896ee40cc4d90567f091efbe0fc9
Cr-Commit-Position: refs/heads/master@{#10286}