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

Issue 726819 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 647283



Sign in to add a comment

Count devices shown in the discovered printers dialog

Project Member Reported by skau@chromium.org, May 26 2017

Issue description

To help understand user's experience with discovered printers, track the number shown in the dialog using UMA.

go/chromeos-cups-metrics
 

Comment 2 by skau@chromium.org, Jun 6 2017

Labels: Merge-Request-60
Status: Fixed (was: Started)
Request merge to M60.  Logging verified on Canary 61.0.3119.0 using a Kip.
Project Member

Comment 3 by sheriffbot@chromium.org, Jun 6 2017

Labels: -Merge-Request-60 Hotlist-Merge-Approved Merge-Approved-60
Your change meets the bar and is auto-approved for M60. Please go ahead and merge the CL to branch 3112 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 6 2017

Labels: -merge-approved-60 merge-merged-3112
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/c86d7012ddedff5dd300906c9ec1ce9bccc9aae3

commit c86d7012ddedff5dd300906c9ec1ce9bccc9aae3
Author: Sean Kau <skau@chromium.org>
Date: Tue Jun 06 21:13:03 2017

Add Printing.CUPS.PrintersDiscovered to our metrics.

In order to understand how many printers users are seeing in the
discovered printers dialog, track it using UMA.  The metric is recorded
when the discoverer declares scanning to be complete.

BUG= 726819 

Review-Url: https://codereview.chromium.org/2904243003
Cr-Original-Commit-Position: refs/heads/master@{#475649}
Review-Url: https://codereview.chromium.org/2926653003 .
Cr-Commit-Position: refs/branch-heads/3112@{#207}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}

[modify] https://crrev.com/c86d7012ddedff5dd300906c9ec1ce9bccc9aae3/chrome/browser/chromeos/printing/printer_discoverer.cc
[modify] https://crrev.com/c86d7012ddedff5dd300906c9ec1ce9bccc9aae3/chrome/browser/chromeos/printing/printer_discoverer.h
[modify] https://crrev.com/c86d7012ddedff5dd300906c9ec1ce9bccc9aae3/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.cc
[modify] https://crrev.com/c86d7012ddedff5dd300906c9ec1ce9bccc9aae3/chrome/browser/ui/webui/settings/chromeos/cups_printers_handler.h
[modify] https://crrev.com/c86d7012ddedff5dd300906c9ec1ce9bccc9aae3/tools/metrics/histograms/histograms.xml

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

Status: Archived (was: Fixed)

Sign in to add a comment