Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(260)

Issue 2685623002: [Payments] Add completion metrics based on user had suggestions for all. (Closed)

Created:
3 years, 10 months ago by sebsg
Modified:
3 years, 10 months ago
CC:
chromium-reviews, asvitkine+watch_chromium.org, rouslan+payments_chromium.org, sebsg+paymentswatch_chromium.org, gogerald+paymentswatch_chromium.org, agrieve+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

[Payments] Add completion metrics based on user had suggestions for all. Add metrics to mesure the impact of the user having suggestions for all the requested sections on whether that user completes the flow. BUG=689551 Review-Url: https://codereview.chromium.org/2685623002 Cr-Commit-Position: refs/heads/master@{#449319} Committed: https://chromium.googlesource.com/chromium/src/+/6022be1d30e10c1282502054bb9268f1b78e13c1

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+171 lines, -27 lines) Patch
M chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestJourneyLogger.java View 5 chunks +28 lines, -8 lines 0 comments Download
M chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestCanMakePaymentMetricsTest.java View 6 chunks +6 lines, -6 lines 0 comments Download
M chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestJourneyLoggerUnitTest.java View 10 chunks +117 lines, -13 lines 0 comments Download
M tools/metrics/histograms/histograms.xml View 1 chunk +20 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (4 generated)
sebsg
Hi! Rouslan@ could you please take a look at all the files? jwd@ could you ...
3 years, 10 months ago (2017-02-07 16:51:20 UTC) #2
please use gerrit instead
lgtm
3 years, 10 months ago (2017-02-08 18:28:18 UTC) #3
jwd
lgtm
3 years, 10 months ago (2017-02-09 15:53:11 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2685623002/1
3 years, 10 months ago (2017-02-09 15:53:50 UTC) #6
commit-bot: I haz the power
3 years, 10 months ago (2017-02-09 16:50:49 UTC) #9
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/6022be1d30e10c1282502054bb92...

Powered by Google App Engine
This is Rietveld 408576698