New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 742595 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Jul 2017
Cc:
Components:
EstimatedDays: 3
NextAction: ----
OS: Chrome
Pri: 1
Type: Feature



Sign in to add a comment

Add UMA for Play Store App Search

Project Member Reported by hejq@chromium.org, Jul 13 2017

Issue description

Project Member

Comment 1 by bugdroid1@chromium.org, Jul 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/36d38a29dc4a584117af2fdbc29368f5c13c11d4

commit 36d38a29dc4a584117af2fdbc29368f5c13c11d4
Author: Jiaquan He <hejq@google.com>
Date: Thu Jul 27 16:07:08 2017

Add UMA for Play Store app search.

- Record the round trip time of a Play Store app search query.
- Record user actions of clicking into uninstalled Play Store
  apps and instant apps.
- Record the numbers of returned uninstalled apps, instant apps,
  and their sum for every Play Store app search query.
- Add Play Store app search result opening actions to the
  Apps.AppListSearchResultOpenType histogram.

Also to distinguish canceled Play Store app search requests from
completed ones, this commit contains a mojo API change.

BUG= 742595 

Change-Id: I94d1fcd985f123afdb5837bcd71f4aaf67323813
Reviewed-on: https://chromium-review.googlesource.com/576396
Commit-Queue: Jiaquan He <hejq@google.com>
Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Vadim Tryshev <vadimt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#489951}
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.cc
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.h
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/chrome/browser/ui/app_list/search/arc/arc_playstore_search_result.cc
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/chrome/browser/ui/app_list/search/search_util.h
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/components/arc/common/app.mojom
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/components/arc/test/fake_app_instance.cc
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/tools/metrics/actions/actions.xml
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/36d38a29dc4a584117af2fdbc29368f5c13c11d4/tools/metrics/histograms/histograms.xml

Comment 2 by hejq@chromium.org, Jul 27 2017

Status: Fixed (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 2 2017

Labels: merge-merged-3163
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7f16989eed9fc3a1039eaa3685a1e77109bd5a92

commit 7f16989eed9fc3a1039eaa3685a1e77109bd5a92
Author: Jiaquan He <hejq@google.com>
Date: Wed Aug 02 22:37:20 2017

Add UMA for Play Store app search.

- Record the round trip time of a Play Store app search query.
- Record user actions of clicking into uninstalled Play Store
  apps and instant apps.
- Record the numbers of returned uninstalled apps, instant apps,
  and their sum for every Play Store app search query.
- Add Play Store app search result opening actions to the
  Apps.AppListSearchResultOpenType histogram.

Also to distinguish canceled Play Store app search requests from
completed ones, this commit contains a mojo API change.

BUG= 742595 
TBR=hejq@google.com

(cherry picked from commit 36d38a29dc4a584117af2fdbc29368f5c13c11d4)

Change-Id: I94d1fcd985f123afdb5837bcd71f4aaf67323813
Reviewed-on: https://chromium-review.googlesource.com/576396
Commit-Queue: Jiaquan He <hejq@google.com>
Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Vadim Tryshev <vadimt@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#489951}
Reviewed-on: https://chromium-review.googlesource.com/596690
Cr-Commit-Position: refs/branch-heads/3163@{#249}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.cc
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.h
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/chrome/browser/ui/app_list/search/arc/arc_playstore_search_result.cc
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/chrome/browser/ui/app_list/search/search_util.h
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/components/arc/common/app.mojom
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/components/arc/test/fake_app_instance.cc
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/tools/metrics/actions/actions.xml
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/7f16989eed9fc3a1039eaa3685a1e77109bd5a92/tools/metrics/histograms/histograms.xml

Comment 4 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment