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

Issue 761155 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Components:
EstimatedDays: 1
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

Add Play Store request result histogram

Project Member Reported by hejq@chromium.org, Aug 31 2017

Issue description

Record the state of each Play Store app search query: SUCCESS, CANCELED, ERROR.
 

Comment 1 by hejq@chromium.org, Aug 31 2017

Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Sep 1 2017

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

commit 440a5fe8a0564bc37d48f9bf9e206cf8e71e4b4c
Author: Jiaquan He <hejq@google.com>
Date: Fri Sep 01 21:19:43 2017

Add Play Store query state histogram.

Bug:  761155 
Change-Id: I2d6f1e7aa8875e102b823818a6d275a3302c454b
Reviewed-on: https://chromium-review.googlesource.com/646831
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Jiaquan He <hejq@google.com>
Cr-Commit-Position: refs/heads/master@{#499301}
[modify] https://crrev.com/440a5fe8a0564bc37d48f9bf9e206cf8e71e4b4c/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.cc
[modify] https://crrev.com/440a5fe8a0564bc37d48f9bf9e206cf8e71e4b4c/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/440a5fe8a0564bc37d48f9bf9e206cf8e71e4b4c/tools/metrics/histograms/histograms.xml

Comment 3 by hejq@chromium.org, Sep 1 2017

Status: Fixed (was: Started)
Project Member

Comment 4 by bugdroid1@chromium.org, Sep 2 2017

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

commit e277e124afc7c3c247804cc92ff0106fdcb6f4be
Author: Vadim Tryshev <vadimt@google.com>
Date: Sat Sep 02 03:22:45 2017

Add Play Store query state histogram.

TBR=hejq@google.com

(cherry picked from commit 440a5fe8a0564bc37d48f9bf9e206cf8e71e4b4c)

Bug:  761155 
Change-Id: I2d6f1e7aa8875e102b823818a6d275a3302c454b
Reviewed-on: https://chromium-review.googlesource.com/646831
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Jiaquan He <hejq@google.com>
Cr-Original-Commit-Position: refs/heads/master@{#499301}
Reviewed-on: https://chromium-review.googlesource.com/648666
Reviewed-by: Vadim Tryshev <vadimt@chromium.org>
Cr-Commit-Position: refs/branch-heads/3163@{#1086}
Cr-Branched-From: ff259bab28b35d242e10186cd63af7ed404fae0d-refs/heads/master@{#488528}
[modify] https://crrev.com/e277e124afc7c3c247804cc92ff0106fdcb6f4be/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.cc
[modify] https://crrev.com/e277e124afc7c3c247804cc92ff0106fdcb6f4be/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/e277e124afc7c3c247804cc92ff0106fdcb6f4be/tools/metrics/histograms/histograms.xml

Project Member

Comment 5 by bugdroid1@chromium.org, Sep 2 2017

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

commit b1637f85c6a288a3765fde45730320d9cf49ac64
Author: Vadim Tryshev <vadimt@google.com>
Date: Sat Sep 02 03:25:20 2017

Add Play Store query state histogram.

TBR=hejq@google.com

(cherry picked from commit 440a5fe8a0564bc37d48f9bf9e206cf8e71e4b4c)

Bug:  761155 
Change-Id: I2d6f1e7aa8875e102b823818a6d275a3302c454b
Reviewed-on: https://chromium-review.googlesource.com/646831
Reviewed-by: Steven Holte <holte@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Jiaquan He <hejq@google.com>
Cr-Original-Commit-Position: refs/heads/master@{#499301}
Reviewed-on: https://chromium-review.googlesource.com/648667
Reviewed-by: Vadim Tryshev <vadimt@chromium.org>
Cr-Commit-Position: refs/branch-heads/3202@{#12}
Cr-Branched-From: fa6a5d87adff761bc16afc5498c3f5944c1daa68-refs/heads/master@{#499098}
[modify] https://crrev.com/b1637f85c6a288a3765fde45730320d9cf49ac64/chrome/browser/ui/app_list/search/arc/arc_playstore_search_provider.cc
[modify] https://crrev.com/b1637f85c6a288a3765fde45730320d9cf49ac64/tools/metrics/histograms/enums.xml
[modify] https://crrev.com/b1637f85c6a288a3765fde45730320d9cf49ac64/tools/metrics/histograms/histograms.xml

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

Status: Archived (was: Fixed)

Comment 7 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment