Merge "[UserClassifier] Fix the classification for M58 condensed NTP..."

Merge "[UserClassifier] Fix the classification for M58 condensed NTP..."

This is a merge of https://codereview.chromium.org/2732243002 into M58
(branch 3029).

There were no conflicts.

=== Original description ===

Previously, UserClassifier used suggestions impressions as activity
signal. This breaks for condensed NTP when impressions are tied to
opening an NTP.

This CL replaces this signal by clicks. Clicks were previously recorded
and not used. After this CL, impressions will stay being recorded. The
reason is that with further changes in the UI, we may need it again. There is a TODO to remove this metric if not used.

This CL also changes default values for activity metrics. The reason is
to make sure that a new user is classified in the middle class.

Lastly, the CL relaxes the thresholds. Currently, we have 67% rare, 33% active users, 1% active consumers. The aim is to move closer towards 55% / 40% / 5%.

BUG=699021
Review-Url: https://codereview.chromium.org/2732243002
Cr-Commit-Position: refs/heads/master@{#455734}
(cherry picked from commit f215f20a3eba699ac39d8bc5c0e4cf8e0d9b0610)

Review-Url: https://codereview.chromium.org/2748053002 .
Cr-Commit-Position: refs/branch-heads/3029@{#183}
Cr-Branched-From: 939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed: https://chromium.googlesource.com/chromium/src/+/5b1922716b0af3cb32f16d329d0cbc131b25c46a

[vitaliii, 2017-03-14 07:56:20]

Description was changed from

==========
Merge "[UserClassifier] Fix the classification for M58 condensed NTP..."

This is a merge of https://codereview.chromium.org/2732243002 into M58
(branch 3029).

There were no conflicts.

=== Original description ===

Previously, UserClassifier used suggestions impressions as activity
signal. This breaks for condensed NTP when impressions are tied to
opening an NTP.

This CL replaces this signal by clicks. Clicks were previously recorded
and not used. After this CL, impressions will stay being recorded. The
reason is that with further changes in the UI, we may need it again. There is a
TODO to remove this metric if not used.

This CL also changes default values for activity metrics. The reason is
to make sure that a new user is classified in the middle class.

Lastly, the CL relaxes the thresholds. Currently, we have 67% rare, 33% active
users, 1% active consumers. The aim is to move closer towards 55% / 40% / 5%.

BUG=699021

Review-Url: https://codereview.chromium.org/2732243002
Cr-Commit-Position: refs/heads/master@{#455734}
(cherry picked from commit f215f20a3eba699ac39d8bc5c0e4cf8e0d9b0610)
==========

to

==========
Merge "[UserClassifier] Fix the classification for M58 condensed NTP..."

This is a merge of https://codereview.chromium.org/2732243002 into M58
(branch 3029).

There were no conflicts.

=== Original description ===

Previously, UserClassifier used suggestions impressions as activity
signal. This breaks for condensed NTP when impressions are tied to
opening an NTP.

This CL replaces this signal by clicks. Clicks were previously recorded
and not used. After this CL, impressions will stay being recorded. The
reason is that with further changes in the UI, we may need it again. There is a
TODO to remove this metric if not used.

This CL also changes default values for activity metrics. The reason is
to make sure that a new user is classified in the middle class.

Lastly, the CL relaxes the thresholds. Currently, we have 67% rare, 33% active
users, 1% active consumers. The aim is to move closer towards 55% / 40% / 5%.

BUG=699021

Review-Url: https://codereview.chromium.org/2732243002
Cr-Commit-Position: refs/heads/master@{#455734}
(cherry picked from commit f215f20a3eba699ac39d8bc5c0e4cf8e0d9b0610)

Review-Url: https://codereview.chromium.org/2748053002 .
Cr-Commit-Position: refs/branch-heads/3029@{#183}
Cr-Branched-From:
939b32ee5ba05c396eef3fd992822fcca9a2e262-refs/heads/master@{#454471}
Committed:
https://chromium.googlesource.com/chromium/src/+/5b1922716b0af3cb32f16d329d0c...
==========

[vitaliii, 2017-03-14 07:56:21]

Committed patchset #1 (id:1) manually as
5b1922716b0af3cb32f16d329d0cbc131b25c46a.