Should only record abort metrics if the Payment Request was shown to the user. |
||||||
Issue description^^^
,
Feb 9 2017
,
Feb 13 2017
,
Feb 13 2017
Your change meets the bar and is auto-approved for M57. Please go ahead and merge the CL to branch 2987 manually. Please contact milestone owner if you have questions. Owners: amineer@(clank), cmasso@(bling), ketakid@(cros), govind@(desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Feb 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8754e89ee5a417f3fc2406beee9b002a20bd393c commit 8754e89ee5a417f3fc2406beee9b002a20bd393c Author: sebsg <sebsg@chromium.org> Date: Tue Feb 14 15:44:15 2017 [Merge-57] Only record abort metrics if the Payment Request was shown. Also move the metrics of event that prevent showing the payment request to their separate bucket. BUG= 689111 , 689232 Review-Url: https://codereview.chromium.org/2678903002 Cr-Commit-Position: refs/heads/master@{#449325} (cherry picked from commit 752b06a473dd6f9a4072c9412431a90aa70f6f81) Review-Url: https://codereview.chromium.org/2692853005 . Cr-Commit-Position: refs/branch-heads/2987@{#498} Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943} [modify] https://crrev.com/8754e89ee5a417f3fc2406beee9b002a20bd393c/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java [modify] https://crrev.com/8754e89ee5a417f3fc2406beee9b002a20bd393c/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestMetrics.java [modify] https://crrev.com/8754e89ee5a417f3fc2406beee9b002a20bd393c/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestJourneyLoggerTest.java [modify] https://crrev.com/8754e89ee5a417f3fc2406beee9b002a20bd393c/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java [modify] https://crrev.com/8754e89ee5a417f3fc2406beee9b002a20bd393c/tools/metrics/histograms/histograms.xml
,
Jun 27 2017
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by bugdroid1@chromium.org
, Feb 9 2017