Make logging of user actions consistent with histograms for OTR.

Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in
counts when compared to histograms. This CL makes them consistent
by making user actions be logged regardless of the OTR state.

BUG=604001
TBR=sgurun@chromium.org, wez@chromium.org, rohitrao@chromium.org

Committed: https://crrev.com/7573bd00c132474546bb4f23b123c2612fce3731
Cr-Commit-Position: refs/heads/master@{#422133}

[Alexei Svitkine (slow), 2016-09-29 14:41:58]

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

[commit-bot: I haz the power, 2016-09-29 14:42:12]

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

[Alexei Svitkine (slow), 2016-09-29 14:54:46]

Description was changed from

==========
Make logging of user actions consistent with histograms for OTR.

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

to

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in counts
when compared to histograms. This CL makes them consistent by
making user actions be logged regardless of the OTR state.

BUG=604001
==========

[Alexei Svitkine (slow), 2016-09-29 14:55:01]

Description was changed from

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in counts
when compared to histograms. This CL makes them consistent by
making user actions be logged regardless of the OTR state.

BUG=604001
==========

to

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in
counts when compared to histograms. This CL makes them consistent
by making user actions be logged regardless of the OTR state.

BUG=604001
==========

[Alexei Svitkine (slow), 2016-09-29 14:55:18]

asvitkine@chromium.org changed reviewers:
+ rkaplow@chromium.org

[commit-bot: I haz the power, 2016-09-29 14:59:05]

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

[commit-bot: I haz the power, 2016-09-29 14:59:06]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)

[Alexei Svitkine (slow), 2016-09-29 14:59:26]

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

[commit-bot: I haz the power, 2016-09-29 14:59:39]

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

[Alexei Svitkine (slow), 2016-09-29 14:59:44]

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

[commit-bot: I haz the power, 2016-09-29 14:59:51]

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, 2016-09-29 16:06:59]

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

[commit-bot: I haz the power, 2016-09-29 16:07:00]

Dry run: This issue passed the CQ dry run.

[Alexei Svitkine (slow), 2016-09-29 16:27:38]

Should be ready for review now.

[rkaplow, 2016-09-29 16:59:35]

lgtm

[Alexei Svitkine (slow), 2016-09-30 15:59:15]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2016-09-30 15:59: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, 2016-09-30 16:04:57]

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

[commit-bot: I haz the power, 2016-09-30 16:04:59]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Alexei Svitkine (slow), 2016-09-30 16:24:54]

Description was changed from

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in
counts when compared to histograms. This CL makes them consistent
by making user actions be logged regardless of the OTR state.

BUG=604001
==========

to

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in
counts when compared to histograms. This CL makes them consistent
by making user actions be logged regardless of the OTR state.

BUG=604001
TBR=sgurun@chromium.org, wez@chromium.org, rohitrao@chromium.org
==========

[Alexei Svitkine (slow), 2016-09-30 16:25:53]

asvitkine@chromium.org changed reviewers:
+ rohitrao@chromium.org, sgurun@chromium.org, wez@chromium.org

[Alexei Svitkine (slow), 2016-09-30 16:25:53]

+TBR=sgurun@chromium.org, wez@chromium.org, rohitrao@chromium.org

for changes in MetricsServiceClient implementations in android_webview, blimp
and ios

[Alexei Svitkine (slow), 2016-09-30 16:26:06]

The CQ bit was checked by asvitkine@chromium.org

[commit-bot: I haz the power, 2016-09-30 16:26:31]

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, 2016-09-30 17:00:25]

Description was changed from

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in
counts when compared to histograms. This CL makes them consistent
by making user actions be logged regardless of the OTR state.

BUG=604001
TBR=sgurun@chromium.org, wez@chromium.org, rohitrao@chromium.org
==========

to

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in
counts when compared to histograms. This CL makes them consistent
by making user actions be logged regardless of the OTR state.

BUG=604001
TBR=sgurun@chromium.org, wez@chromium.org, rohitrao@chromium.org
==========

[commit-bot: I haz the power, 2016-09-30 17:00:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-09-30 17:06:28]

Description was changed from

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in
counts when compared to histograms. This CL makes them consistent
by making user actions be logged regardless of the OTR state.

BUG=604001
TBR=sgurun@chromium.org, wez@chromium.org, rohitrao@chromium.org
==========

to

==========
Make logging of user actions consistent with histograms for OTR.

Previously, we would not record user actions in UMA when an OTR
(Incognito) window was active. However, since user actions don't
contain any PII it was just causing a confusing discrepancy in
counts when compared to histograms. This CL makes them consistent
by making user actions be logged regardless of the OTR state.

BUG=604001
TBR=sgurun@chromium.org, wez@chromium.org, rohitrao@chromium.org

Committed: https://crrev.com/7573bd00c132474546bb4f23b123c2612fce3731
Cr-Commit-Position: refs/heads/master@{#422133}
==========

[commit-bot: I haz the power, 2016-09-30 17:06:30]

Patchset 2 (id:??) landed as
https://crrev.com/7573bd00c132474546bb4f23b123c2612fce3731
Cr-Commit-Position: refs/heads/master@{#422133}