Set up switches for user language detection and translation events.

Set up switches for user language detection and translation events.

These two switches control logging language detection and translation
events behavior. However, we still rely on SyncUserEvents as a
centralized gateway to logging.


BUG=722679, 728491
Review-Url: https://codereview.chromium.org/2932603002
Cr-Commit-Position: refs/heads/master@{#479247}
Committed: https://chromium.googlesource.com/chromium/src/+/cdce2652bc147496e6d53348d80f21c8d7e7dec6

[renjieliu1, 2017-06-08 00:46:18]

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

[commit-bot: I haz the power, 2017-06-08 00:46:34]

Dry run: CQ is trying da patch.

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

[renjieliu1, 2017-06-08 00:48:05]

Description was changed from

==========
Set up switches for user language detection and translation events.


BUG=722679,728491
==========

to

==========
Set up switches for user language detection and translation events.

These two switches control logging language detection and translation
events behavior. However, we still rely on SyncUserEvents as a
centralized gateway to logging.


BUG=722679,728491
==========

[renjieliu1, 2017-06-08 00:48:05]

renjieliu@chromium.org changed reviewers:
+ groby@chromium.org, napper@chromium.org

[commit-bot: I haz the power, 2017-06-08 02:24:56]

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

[commit-bot: I haz the power, 2017-06-08 02:24:57]

Dry run: This issue passed the CQ dry run.

[napper, 2017-06-09 17:38:33]

lgtm but note we are planning to only have a single launch (i.e., both detection
/ translation at the same time). But splitting it into two flags seems okay to
me in case some issue comes up with either of the logging types.

[renjieliu1, 2017-06-12 23:46:36]

On 2017/06/09 17:38:33, napper wrote:
> lgtm but note we are planning to only have a single launch (i.e., both
detection
> / translation at the same time). But splitting it into two flags seems okay to
> me in case some issue comes up with either of the logging types.

sgtm :) I will keep two flags in case something wrong. Thanks.

[renjieliu1, 2017-06-13 00:06:44]

renjieliu@chromium.org changed reviewers:
+ skym@chromium.org

[groby-ooo-7-16, 2017-06-13 00:28:15]

lgtm

[skym, 2017-06-13 18:44:13]

lgtm

[renjieliu1, 2017-06-14 00:18:04]

The CQ bit was checked by renjieliu@chromium.org

[commit-bot: I haz the power, 2017-06-14 00:18:30]

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-14 01:41:20]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1497399484379380, "parent_rev":
"7f4d857166a63d68e02e9ef769985d7f1cc8f2c2", "commit_rev":
"cdce2652bc147496e6d53348d80f21c8d7e7dec6"}

[commit-bot: I haz the power, 2017-06-14 01:41:41]

Description was changed from

==========
Set up switches for user language detection and translation events.

These two switches control logging language detection and translation
events behavior. However, we still rely on SyncUserEvents as a
centralized gateway to logging.


BUG=722679,728491
==========

to

==========
Set up switches for user language detection and translation events.

These two switches control logging language detection and translation
events behavior. However, we still rely on SyncUserEvents as a
centralized gateway to logging.


BUG=722679,728491

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

[commit-bot: I haz the power, 2017-06-14 01:41:43]

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