Reland "Add enable_ipc_logging build argument"

Reland "Add enable_ipc_logging build argument (patchset #2 id:20001 of https://codereview.chromium.org/2770653002/ )"

It introduced a build flake due to assorted missing dependencies
on //ipc and it was reverted in https://codereview.chromium.org/2768403002.
The public dependencies on //ipc are fixed in https://codereview.chromium.org/2767193005.

Original issue's description:
> Add enable_ipc_logging build argument
>
> Implement a build option to enable the IPC logging system
> in release builds. It's useful to save time and resources when
> debugging IPC communication (e.g. in automated testing
> environments).
>
> It also turns IPC_MESSAGE_LOG_ENABLED macro to a build flag.
>
> BUG=
>
> Review-Url: https://codereview.chromium.org/2770653002
> Cr-Commit-Position: refs/heads/master@{#459094}
> Committed: https://chromium.googlesource.com/chromium/src/+/76890ec11e7402591d1c9add533d4e61c30f8e08

BUG=

Review-Url: https://codereview.chromium.org/2777983005
Cr-Commit-Position: refs/heads/master@{#460152}
Committed: https://chromium.googlesource.com/chromium/src/+/b39d83360c64113ff17e64169c49e1a83329b61e

[Szabolcs David, 2017-03-28 11:47:50]

Description was changed from

==========
Reland "Add enable_ipc_logging build argument (patchset #2 id:20001 of
https://codereview.chromium.org/2770653002/ )"

It introduced a build flake due to assorted missing dependencies
on //ipc and it was reverted in https://codereview.chromium.org/2768403002.
The public dependencies on //ipc are fixed in
https://codereview.chromium.org/2767193005.

Original issue's description:
> Add enable_ipc_logging build argument
>
> Implement a build option to enable the IPC logging system
> in release builds. It's useful to save time and resources when
> debugging IPC communication (e.g. in automated testing
> environments).
>
> It also turns IPC_MESSAGE_LOG_ENABLED macro to a build flag.
>
> BUG=
>
> Review-Url: https://codereview.chromium.org/2770653002
> Cr-Commit-Position: refs/heads/master@{#459094}
> Committed:
https://chromium.googlesource.com/chromium/src/+/76890ec11e7402591d1c9add533d...

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

to

==========
Reland "Add enable_ipc_logging build argument (patchset #2 id:20001 of
https://codereview.chromium.org/2770653002/ )"

It introduced a build flake due to assorted missing dependencies
on //ipc and it was reverted in https://codereview.chromium.org/2768403002.
The public dependencies on //ipc are fixed in
https://codereview.chromium.org/2767193005.

Original issue's description:
> Add enable_ipc_logging build argument
>
> Implement a build option to enable the IPC logging system
> in release builds. It's useful to save time and resources when
> debugging IPC communication (e.g. in automated testing
> environments).
>
> It also turns IPC_MESSAGE_LOG_ENABLED macro to a build flag.
>
> BUG=
>
> Review-Url: https://codereview.chromium.org/2770653002
> Cr-Commit-Position: refs/heads/master@{#459094}
> Committed:
https://chromium.googlesource.com/chromium/src/+/76890ec11e7402591d1c9add533d...

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

[Szabolcs David, 2017-03-28 11:48:36]

The CQ bit was checked by davidsz@inf.u-szeged.hu to run a CQ dry run

[commit-bot: I haz the power, 2017-03-28 11:48:47]

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

[commit-bot: I haz the power, 2017-03-28 11:48:48]

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

[commit-bot: I haz the power, 2017-03-28 11:48:49]

Dry run: No L-G-T-M from a valid reviewer yet.
CQ run can only be started once the patch has received an L-G-T-M from a full
committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,_not_ a
full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[Szabolcs David, 2017-03-28 11:57:44]

davidsz@inf.u-szeged.hu changed reviewers:
+ jam@chromium.org, kenrb@chromium.org, rockot@chromium.org

[Szabolcs David, 2017-03-28 11:58:05]

Can we try it again? :)

[jam, 2017-03-28 15:36:42]

rs lgtm

[Szabolcs David, 2017-03-28 15:38:56]

The CQ bit was checked by davidsz@inf.u-szeged.hu to run a CQ dry run

[kenrb, 2017-03-28 15:39:07]

lgtm

[commit-bot: I haz the power, 2017-03-28 15:39:40]

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

[Ken Rockot(use gerrit already), 2017-03-28 15:40:47]

lgtm

[Szabolcs David, 2017-03-28 15:41:06]

The CQ bit was unchecked by davidsz@inf.u-szeged.hu

[Szabolcs David, 2017-03-28 15:41:14]

The CQ bit was checked by davidsz@inf.u-szeged.hu

[commit-bot: I haz the power, 2017-03-28 15:42:08]

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

[commit-bot: I haz the power, 2017-03-28 17:24:27]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1490715674723010, "parent_rev":
"2753f4dd3d49fddb16c538feddb125c92cc0fe23", "commit_rev":
"b39d83360c64113ff17e64169c49e1a83329b61e"}

[commit-bot: I haz the power, 2017-03-28 17:25:13]

Description was changed from

==========
Reland "Add enable_ipc_logging build argument (patchset #2 id:20001 of
https://codereview.chromium.org/2770653002/ )"

It introduced a build flake due to assorted missing dependencies
on //ipc and it was reverted in https://codereview.chromium.org/2768403002.
The public dependencies on //ipc are fixed in
https://codereview.chromium.org/2767193005.

Original issue's description:
> Add enable_ipc_logging build argument
>
> Implement a build option to enable the IPC logging system
> in release builds. It's useful to save time and resources when
> debugging IPC communication (e.g. in automated testing
> environments).
>
> It also turns IPC_MESSAGE_LOG_ENABLED macro to a build flag.
>
> BUG=
>
> Review-Url: https://codereview.chromium.org/2770653002
> Cr-Commit-Position: refs/heads/master@{#459094}
> Committed:
https://chromium.googlesource.com/chromium/src/+/76890ec11e7402591d1c9add533d...

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

to

==========
Reland "Add enable_ipc_logging build argument (patchset #2 id:20001 of
https://codereview.chromium.org/2770653002/ )"

It introduced a build flake due to assorted missing dependencies
on //ipc and it was reverted in https://codereview.chromium.org/2768403002.
The public dependencies on //ipc are fixed in
https://codereview.chromium.org/2767193005.

Original issue's description:
> Add enable_ipc_logging build argument
>
> Implement a build option to enable the IPC logging system
> in release builds. It's useful to save time and resources when
> debugging IPC communication (e.g. in automated testing
> environments).
>
> It also turns IPC_MESSAGE_LOG_ENABLED macro to a build flag.
>
> BUG=
>
> Review-Url: https://codereview.chromium.org/2770653002
> Cr-Commit-Position: refs/heads/master@{#459094}
> Committed:
https://chromium.googlesource.com/chromium/src/+/76890ec11e7402591d1c9add533d...

BUG=

Review-Url: https://codereview.chromium.org/2777983005
Cr-Commit-Position: refs/heads/master@{#460152}
Committed:
https://chromium.googlesource.com/chromium/src/+/b39d83360c64113ff17e64169c49...
==========

[commit-bot: I haz the power, 2017-03-28 17:25:15]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/b39d83360c64113ff17e64169c49...

[jwd, 2017-03-28 21:10:03]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2776393003/ by jwd@chromium.org.

The reason for reverting is: This still appears to be causing a build flake on
https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

log:
https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Ftryserver.chromium.linux....