New issue
Advanced search Search tips

Issue 740546 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Add metrics to analyse completion ratio based on if the user had complete suggestions.

Project Member Reported by se...@chromium.org, Jul 10 2017

Issue description

^^^
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 19 2017

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

commit ce700484dd990aea0992dff2bf36e6e2cedbe98e
Author: sebsg <sebsg@chromium.org>
Date: Wed Jul 19 23:25:38 2017

[Payments] Add metric to track if user had complete suggestions.

Also change section metrics from CreditCards to PaymentMethod
(Now include payment apps). Which removes the need for the
hadInitialFormOfPayment metrics.

Bug:  740546 
Change-Id: If2e67b66e4c309654ae1ade64d51f1d070e13d1a
Reviewed-on: https://chromium-review.googlesource.com/565340
Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Mathieu Perreault <mathp@chromium.org>
Commit-Queue: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488033}
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/ContactDetailsSection.java
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/android/java/src/org/chromium/chrome/browser/payments/ui/PaymentOption.java
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestContactDetailsSectionUnitTest.java
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestJourneyLoggerTest.java
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestCommon.java
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestRule.java
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/browser/payments/android/journey_logger_android.cc
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/browser/payments/android/journey_logger_android.h
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/chrome/browser/ui/views/payments/payment_request_journey_logger_browsertest.cc
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/components/payments/content/payment_request_state.cc
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/components/payments/core/journey_logger.cc
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/components/payments/core/journey_logger.h
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/components/payments/core/journey_logger_unittest.cc
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/components/test/data/payments/metrics.js
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/components/test/data/payments/payment_request_metrics_test.html
[modify] https://crrev.com/ce700484dd990aea0992dff2bf36e6e2cedbe98e/tools/metrics/histograms/histograms.xml

Comment 2 by se...@chromium.org, Jul 20 2017

Status: Fixed (was: Started)

Sign in to add a comment