📰 Add user action for the first time the UI is shown

[Suggestions] Add user action for the first time the UI is shown

BUG=717704
Review-Url: https://codereview.chromium.org/2878773002
Cr-Commit-Position: refs/heads/master@{#473584}
Committed: https://chromium.googlesource.com/chromium/src/+/f0a16200825ea532a475b9169f9fce6c040793d5

[dgn, 2017-05-11 19:54:51]

dgn@chromium.org changed reviewers:
+ bauerb@chromium.org

[dgn, 2017-05-11 19:54:52]

PTAL

[Bernhard Bauer, 2017-05-12 09:41:12]

lgtm

https://codereview.chromium.org/2878773002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsMetrics.java
(right):

https://codereview.chromium.org/2878773002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsMetrics.java:20:
RecordUserAction.record("Suggestions.FirstTimeSurfaceVisible");
I think would record that after Suggestions.SurfaceVisible, so that we can see
from the sequence visualization how many opens are first-time.

[dgn, 2017-05-12 13:14:58]

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

[commit-bot: I haz the power, 2017-05-12 13:15:13]

Dry run: CQ is trying da patch.

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

[dgn, 2017-05-12 13:15:27]

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

[dgn, 2017-05-12 13:15:28]

Thanks!

asvitkine@ PTAL at actions.xml

https://codereview.chromium.org/2878773002/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsMetrics.java
(right):

https://codereview.chromium.org/2878773002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsMetrics.java:20:
RecordUserAction.record("Suggestions.FirstTimeSurfaceVisible");
On 2017/05/12 09:41:12, Bernhard Bauer wrote:
> I think would record that after Suggestions.SurfaceVisible, so that we can see
> from the sequence visualization how many opens are first-time.

As discussed offline, we can display sequence starting from an action in both
directions so we can still have the information of how many opens are first
time. Also doing it in this direction avoids further dividing the sequences
starting from SurfaceVisible

[commit-bot: I haz the power, 2017-05-12 13:30:04]

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

[commit-bot: I haz the power, 2017-05-12 13:30:05]

Dry run: 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...)

[dgn, 2017-05-12 13:57:02]

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

[commit-bot: I haz the power, 2017-05-12 13:57:18]

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-05-12 14:09:31]

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

[commit-bot: I haz the power, 2017-05-12 14:09:32]

Dry run: 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), 2017-05-12 14:19:00]

lgtm

[dgn, 2017-05-22 14:09:36]

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

[commit-bot: I haz the power, 2017-05-22 14:09:52]

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-05-22 14:52:39]

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

[commit-bot: I haz the power, 2017-05-22 14:52:40]

Dry run: This issue passed the CQ dry run.

[dgn, 2017-05-22 15:53:59]

The CQ bit was checked by dgn@chromium.org

[dgn, 2017-05-22 15:54:00]

The patchset sent to the CQ was uploaded after l-g-t-m from bauerb@chromium.org,
asvitkine@chromium.org
Link to the patchset: https://codereview.chromium.org/2878773002/#ps60001
(title: "rebase, remove not_user_triggered")

[commit-bot: I haz the power, 2017-05-22 15:54:23]

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 16:02:43]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1495468439690470,
"parent_rev": "693c21e381f14e84e67137c2307e1c83a24923fe", "commit_rev":
"f0a16200825ea532a475b9169f9fce6c040793d5"}

[commit-bot: I haz the power, 2017-05-22 16:02:56]

Description was changed from

==========
[Suggestions] Add user action for the first time the UI is shown

BUG=717704
==========

to

==========
[Suggestions] Add user action for the first time the UI is shown

BUG=717704

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

[commit-bot: I haz the power, 2017-05-22 16:02:57]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/src/+/f0a16200825ea532a475b9169f9f...