Set the event log in Channel from AudioSendStream

Set the event log in Channel from AudioSendStream. This will re-enable logging of outgoing audio packets.

BUG=webrtc:6195
Committed: https://crrev.com/e035e2d26f9b6ef0a6a5b262fae775080f91bacf
Cr-Commit-Position: refs/heads/master@{#14331}

[terelius, 2016-08-19 11:12:48]

terelius@webrtc.org changed reviewers:
+ ivoc@webrtc.org, stefan@webrtc.org

[terelius, 2016-08-19 11:12:49]

PTAL

[ivoc, 2016-08-19 11:20:42]

LGTM.

[stefan-webrtc, 2016-08-23 10:03:23]

lgtm

[stefan-webrtc, 2016-08-23 10:04:31]

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

[stefan-webrtc, 2016-08-23 10:04:31]

I would however suggest that you get someone who's a webrtc/audio owner to
approve too.

Adding solenberg.

[the sun, 2016-08-23 10:15:29]

lgtm

https://codereview.webrtc.org/2226823003/diff/20001/webrtc/audio/audio_send_s...
File webrtc/audio/audio_send_stream.cc (right):

https://codereview.webrtc.org/2226823003/diff/20001/webrtc/audio/audio_send_s...
webrtc/audio/audio_send_stream.cc:106: channel_proxy_->SetRtcEventLog(nullptr);
super nit: registering methods are called in opposite order of how they are
called in the ctor here. it shouldn't matter, but let's keep it so.

[terelius, 2016-09-19 10:32:45]

Description was changed from

==========
Set the event log in Channel from AudioSendStream

BUG=webrtc:6195
==========

to

==========
Set the event log in Channel from AudioSendStream. This will re-enable logging
of outgoing audio packets.

BUG=webrtc:6195
==========

[terelius, 2016-09-20 13:58:41]

The CQ bit was checked by terelius@webrtc.org

[terelius, 2016-09-20 13:58:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
solenberg@webrtc.org, ivoc@webrtc.org, stefan@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2226823003/#ps40001 (title:
"Registration order and rebase")

[commit-bot: I haz the power, 2016-09-20 13:58:46]

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-09-20 14:39:14]

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

[commit-bot: I haz the power, 2016-09-20 14:39:15]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  android_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_clang_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_clang_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_drmemory_light on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_clang_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_clang_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_gyp_dbg on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_gyp_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)
  win_x64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[terelius, 2016-09-21 07:57:50]

The CQ bit was checked by terelius@webrtc.org

[commit-bot: I haz the power, 2016-09-21 07:57:55]

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-09-21 08:12:44]

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

[commit-bot: I haz the power, 2016-09-21 08:12:45]

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

[terelius, 2016-09-21 12:38:11]

The CQ bit was checked by terelius@webrtc.org

[commit-bot: I haz the power, 2016-09-21 12:38:20]

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-09-21 13:51:50]

Description was changed from

==========
Set the event log in Channel from AudioSendStream. This will re-enable logging
of outgoing audio packets.

BUG=webrtc:6195
==========

to

==========
Set the event log in Channel from AudioSendStream. This will re-enable logging
of outgoing audio packets.

BUG=webrtc:6195
==========

[commit-bot: I haz the power, 2016-09-21 13:51:51]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-21 13:51:55]

Description was changed from

==========
Set the event log in Channel from AudioSendStream. This will re-enable logging
of outgoing audio packets.

BUG=webrtc:6195
==========

to

==========
Set the event log in Channel from AudioSendStream. This will re-enable logging
of outgoing audio packets.

BUG=webrtc:6195

Committed: https://crrev.com/e035e2d26f9b6ef0a6a5b262fae775080f91bacf
Cr-Commit-Position: refs/heads/master@{#14331}
==========

[commit-bot: I haz the power, 2016-09-21 13:51:57]

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