New issue
Advanced search Search tips

Issue 653636 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment

Have UMA Stats for number of masked/unmasked server cards

Project Member Reported by ma...@chromium.org, Oct 6 2016

Issue description

Currently, only have stats for local cards.
 

Comment 2 by ma...@chromium.org, Oct 13 2016

Labels: -M-54 M-55 Merge-Request-55
Hello, requesting merge for M55 for these metrics, thanks!

Comment 3 by ma...@chromium.org, Oct 13 2016

Status: Fixed (was: Started)

Comment 4 by dimu@chromium.org, Oct 13 2016

Labels: -Merge-Request-55 Merge-Approved-55 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M55 (branch: 2883)
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 13 2016

Labels: -merge-approved-55 merge-merged-2883
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bfd60b78145bef610a9eeba4dbdb402198a23966

commit bfd60b78145bef610a9eeba4dbdb402198a23966
Author: Mathieu Perreault <mathp@chromium.org>
Date: Thu Oct 13 14:45:20 2016

[Merge M55][Autofill] Log count histogram for the number of server cards

BUG= 653636 
TEST=AutofillMetrics

Review-Url: https://codereview.chromium.org/2395243002
Cr-Commit-Position: refs/heads/master@{#424053}
(cherry picked from commit 2af1bc9bc29520e5b3893b8c31d008d1b384b296)

Review URL: https://codereview.chromium.org/2415953002 .

Cr-Commit-Position: refs/branch-heads/2883@{#87}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/autofill_metrics.cc
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/autofill_metrics.h
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/autofill_metrics_unittest.cc
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/personal_data_manager.cc
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/personal_data_manager.h
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/tools/metrics/histograms/histograms.xml

Project Member

Comment 6 by bugdroid1@chromium.org, Oct 27 2016

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

commit bfd60b78145bef610a9eeba4dbdb402198a23966
Author: Mathieu Perreault <mathp@chromium.org>
Date: Thu Oct 13 14:45:20 2016

[Merge M55][Autofill] Log count histogram for the number of server cards

BUG= 653636 
TEST=AutofillMetrics

Review-Url: https://codereview.chromium.org/2395243002
Cr-Commit-Position: refs/heads/master@{#424053}
(cherry picked from commit 2af1bc9bc29520e5b3893b8c31d008d1b384b296)

Review URL: https://codereview.chromium.org/2415953002 .

Cr-Commit-Position: refs/branch-heads/2883@{#87}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/autofill_metrics.cc
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/autofill_metrics.h
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/autofill_metrics_unittest.cc
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/personal_data_manager.cc
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/components/autofill/core/browser/personal_data_manager.h
[modify] https://crrev.com/bfd60b78145bef610a9eeba4dbdb402198a23966/tools/metrics/histograms/histograms.xml

Comment 7 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840

Sign in to add a comment