Reland of "Log audio network adapter decisions in event log."

Reland of "Log audio network adapter decisions in event log."

This was originally reviewed https://codereview.webrtc.org/2559953002/

It was reverted due to a bug in the original CL, see https://codereview.webrtc.org/2631703002/

This CL is to fix and reland.

BUG=webrtc:6845
Review-Url: https://codereview.webrtc.org/2644863002
Cr-Commit-Position: refs/heads/master@{#16242}
Committed: https://chromium.googlesource.com/external/webrtc/+/4b7c952376bec4797307b106796c5612d930f206

[minyue-webrtc, 2017-01-19 11:33:10]

Description was changed from

==========
Revert "Revert of Log audio network adapter decisions in event log. (patchset
#14 id:320001 of https://codereview.webrtc.org/2559953002/ )"

This reverts commit 363a29157ad162f48c63e9d6424ff9949af59e4a.

BUG=
==========

to

==========
Reland of "Log audio network adapter decisions in event log."

This was originally reviewed https://codereview.webrtc.org/2559953002/

It was reverted due to a bug in the original CL.

BUG=webrtc:6845
==========

[minyue-webrtc, 2017-01-19 11:33:10]

minyue@webrtc.org changed reviewers:
+ michaelt@webrtc.org, terelius@webrtc.org

[minyue-webrtc, 2017-01-19 11:35:23]

Description was changed from

==========
Reland of "Log audio network adapter decisions in event log."

This was originally reviewed https://codereview.webrtc.org/2559953002/

It was reverted due to a bug in the original CL.

BUG=webrtc:6845
==========

to

==========
Reland of "Log audio network adapter decisions in event log."

This was originally reviewed https://codereview.webrtc.org/2559953002/

It was reverted due to a bug in the original CL, see
https://codereview.webrtc.org/2631703002/

This CL is to fix and reland.

BUG=webrtc:6845
==========

[minyue-webrtc, 2017-01-19 11:35:33]

https://codereview.webrtc.org/2644863002/diff/1/webrtc/logging/BUILD.gn
File webrtc/logging/BUILD.gn (right):

https://codereview.webrtc.org/2644863002/diff/1/webrtc/logging/BUILD.gn#newco...
webrtc/logging/BUILD.gn:64: proto_in_dir = "//"
This change is not needed and is likely the reason for the reversion.

[minyue-webrtc, 2017-01-19 11:36:14]

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

[commit-bot: I haz the power, 2017-01-19 11:36:23]

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

[terelius, 2017-01-19 12:23:40]

https://codereview.webrtc.org/2644863002/diff/20001/webrtc/modules/audio_codi...
File webrtc/modules/audio_coding/BUILD.gn (right):

https://codereview.webrtc.org/2644863002/diff/20001/webrtc/modules/audio_codi...
webrtc/modules/audio_coding/BUILD.gn:891: proto_out_dir = "."
Note that this is similar to the change that (probably) caused the revert. Is
this change needed/intended?

[minyue-webrtc, 2017-01-19 12:46:13]

https://codereview.webrtc.org/2644863002/diff/20001/webrtc/modules/audio_codi...
File webrtc/modules/audio_coding/BUILD.gn (right):

https://codereview.webrtc.org/2644863002/diff/20001/webrtc/modules/audio_codi...
webrtc/modules/audio_coding/BUILD.gn:891: proto_out_dir = "."
On 2017/01/19 12:23:40, terelius wrote:
> Note that this is similar to the change that (probably) caused the revert. Is
> this change needed/intended?

Thanks! This should not be needed either.

[terelius, 2017-01-19 12:59:41]

lgtm

[minyue-webrtc, 2017-01-19 13:09:58]

The CQ bit was checked by minyue@webrtc.org

[commit-bot: I haz the power, 2017-01-19 13:10:06]

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, 2017-01-19 13:14:18]

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

[commit-bot: I haz the power, 2017-01-19 13:14:19]

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

[minyue-webrtc, 2017-01-20 17:21:53]

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

[minyue-webrtc, 2017-01-20 17:22:26]

Add "the sun" for VoE approval.

[minyue-webrtc, 2017-01-21 20:05:39]

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

[commit-bot: I haz the power, 2017-01-21 20:05:50]

Dry run: 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, 2017-01-21 21:18:13]

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

[commit-bot: I haz the power, 2017-01-21 21:18:14]

Dry run: This issue passed the CQ dry run.

[michaelt, 2017-01-23 15:21:52]

lgtm

[the sun, 2017-01-24 08:32:26]

lgtm

[minyue-webrtc, 2017-01-24 09:35:33]

The CQ bit was checked by minyue@webrtc.org

[minyue-webrtc, 2017-01-24 09:35:33]

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

[commit-bot: I haz the power, 2017-01-24 09:35:41]

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

[minyue-webrtc, 2017-01-24 10:49:32]

On 2017/01/24 08:32:26, the sun (OOO until Jan 30th) wrote:
> lgtm

I did not know that you are OOO. Thanks for approving this!

[minyue-webrtc, 2017-01-24 10:49:37]

On 2017/01/24 08:32:26, the sun (OOO until Jan 30th) wrote:
> lgtm

I did not know that you are OOO. Thanks for approving this!

[commit-bot: I haz the power, 2017-01-24 10:52:13]

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

[commit-bot: I haz the power, 2017-01-24 10:52:14]

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

[minyue-webrtc, 2017-01-24 12:36:40]

The CQ bit was checked by minyue@webrtc.org

[commit-bot: I haz the power, 2017-01-24 12:36:43]

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, 2017-01-24 12:55:00]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1485261400672980,
"parent_rev": "35a32700fc9b5d932ddbd528c12f59c3274e4774", "commit_rev":
"4b7c952376bec4797307b106796c5612d930f206"}

[commit-bot: I haz the power, 2017-01-24 12:55:03]

Description was changed from

==========
Reland of "Log audio network adapter decisions in event log."

This was originally reviewed https://codereview.webrtc.org/2559953002/

It was reverted due to a bug in the original CL, see
https://codereview.webrtc.org/2631703002/

This CL is to fix and reland.

BUG=webrtc:6845
==========

to

==========
Reland of "Log audio network adapter decisions in event log."

This was originally reviewed https://codereview.webrtc.org/2559953002/

It was reverted due to a bug in the original CL, see
https://codereview.webrtc.org/2631703002/

This CL is to fix and reland.

BUG=webrtc:6845

Review-Url: https://codereview.webrtc.org/2644863002
Cr-Commit-Position: refs/heads/master@{#16242}
Committed:
https://chromium.googlesource.com/external/webrtc/+/4b7c952376bec4797307b1067...
==========

[commit-bot: I haz the power, 2017-01-24 12:55:04]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/external/webrtc/+/4b7c952376bec4797307b1067...