Disable logging redirect on chromeos.

Disable logging redirect on chromeos.

All logging is now available in /var/log/chrome/chrome.

BUG=chromium:724273
TEST=manual

Review-Url: https://codereview.chromium.org/2901603002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/509327f965c3a6885c1e5bc8abf5ced0478d72d7

[achuithb, 2017-05-22 21:24:00]

The CQ bit was checked by achuith@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-22 21:24:06]

Dry run: CQ is trying da patch.

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

[achuithb, 2017-05-22 21:26:51]

achuith@chromium.org changed reviewers:
+ xiyuan@chromium.org

[achuithb, 2017-05-22 21:26:52]

PTAL, Xiyuan

[commit-bot: I haz the power, 2017-05-22 21:50:19]

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

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

Dry run: This issue passed the CQ dry run.

[xiyuan, 2017-05-22 22:38:01]

lgtm

[achuithb, 2017-05-22 23:43:46]

The CQ bit was checked by achuith@chromium.org

[commit-bot: I haz the power, 2017-05-22 23:43:56]

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-05-22 23:43:57]

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

[commit-bot: I haz the power, 2017-05-22 23:43:58]

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-catapult-committers".
Note that this has nothing to do with OWNERS files.

[achuithb, 2017-05-23 07:34:51]

achuith@chromium.org changed reviewers:
+ jdufault@chromium.org

[achuithb, 2017-05-23 07:34:52]

Jake, could you PTAL

[jdufault, 2017-05-23 17:13:27]

lgtm

[achuithb, 2017-05-23 20:54:21]

The CQ bit was checked by achuith@chromium.org

[commit-bot: I haz the power, 2017-05-23 20:54:34]

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-05-23 20:54:34]

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

[commit-bot: I haz the power, 2017-05-23 20:54:35]

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-catapult-committers".
Note that this has nothing to do with OWNERS files.

[achuithb, 2017-05-23 20:57:11]

achuith@chromium.org changed reviewers:
+ nednguyen@google.com

[achuithb, 2017-05-23 20:57:11]

Ned, could you PTAL

[nednguyen, 2017-05-23 21:08:11]

lgtm

https://codereview.chromium.org/2901603002/diff/1/telemetry/telemetry/interna...
File telemetry/telemetry/internal/backends/chrome/cros_browser_backend.py
(right):

https://codereview.chromium.org/2901603002/diff/1/telemetry/telemetry/interna...
telemetry/telemetry/internal/backends/chrome/cros_browser_backend.py:80: #
Disable chrome logging redirect,
nits: end this sentence with "." & add bug reference for more context.

[achuithb, 2017-06-01 21:37:53]

https://codereview.chromium.org/2901603002/diff/1/telemetry/telemetry/interna...
File telemetry/telemetry/internal/backends/chrome/cros_browser_backend.py
(right):

https://codereview.chromium.org/2901603002/diff/1/telemetry/telemetry/interna...
telemetry/telemetry/internal/backends/chrome/cros_browser_backend.py:80: #
Disable chrome logging redirect,
On 2017/05/23 21:08:11, nednguyen wrote:
> nits: end this sentence with "." & add bug reference for more context.

Done.

[achuithb, 2017-06-01 21:37:57]

The CQ bit was checked by achuith@chromium.org

[achuithb, 2017-06-01 21:37:58]

The patchset sent to the CQ was uploaded after l-g-t-m from xiyuan@chromium.org,
jdufault@chromium.org, nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2901603002/#ps20001
(title: "Ned feedback")

[commit-bot: I haz the power, 2017-06-01 21:38:04]

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-06-01 22:05:49]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1496353077923980,
"parent_rev": "e59e87ea3fbbddbb5ba6174c776fbe61c70e2d00", "commit_rev":
"509327f965c3a6885c1e5bc8abf5ced0478d72d7"}

[commit-bot: I haz the power, 2017-06-01 22:05:53]

Description was changed from

==========
Disable logging redirect on chromeos.

All logging is now available in /var/log/chrome/chrome.

BUG=chromium:724273
TEST=manual
==========

to

==========
Disable logging redirect on chromeos.

All logging is now available in /var/log/chrome/chrome.

BUG=chromium:724273
TEST=manual

Review-Url: https://codereview.chromium.org/2901603002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-06-01 22:05:54]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...