New issue
Advanced search Search tips

Issue 726469 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

Move the logging for checkout flow to native for Payment Request.

Project Member Reported by se...@chromium.org, May 25 2017

Issue description

^^^
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 26 2017

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

commit f8272a2043eeddf86177a2c643215d5060066122
Author: sebsg <sebsg@chromium.org>
Date: Fri May 26 14:32:58 2017

[Payments] Move the logging for checkout flow to native.

BUG= 726469 

Review-Url: https://codereview.chromium.org/2899313002
Cr-Commit-Position: refs/heads/master@{#475001}

[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestMetrics.java
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestShowTwiceTest.java
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestCommon.java
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/chrome/browser/payments/android/journey_logger_android.cc
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/chrome/browser/payments/android/journey_logger_android.h
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/components/payments/content/payment_request.cc
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/components/payments/core/journey_logger.cc
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/components/payments/core/journey_logger.h
[modify] https://crrev.com/f8272a2043eeddf86177a2c643215d5060066122/components/payments/core/journey_logger_unittest.cc

Comment 2 by se...@chromium.org, May 29 2017

Labels: Merge-Request-60
Project Member

Comment 3 by sheriffbot@chromium.org, May 29 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, May 29 2017

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

commit dfb0b7491cf5b22135241076e5af33d59be918fe
Author: sebsg <sebsg@chromium.org>
Date: Mon May 29 21:09:50 2017

Merge-60 [Payments] Move the logging for checkout flow to native.

BUG= 726469 

Review-Url: https://codereview.chromium.org/2899313002
Cr-Original-Commit-Position: refs/heads/master@{#475001}
Review-Url: https://codereview.chromium.org/2914563002 .
Cr-Commit-Position: refs/branch-heads/3112@{#19}
Cr-Branched-From: b6460e24cf59f429d69de255538d0fc7a425ccf9-refs/heads/master@{#474897}

[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/chrome/android/java/src/org/chromium/chrome/browser/payments/JourneyLogger.java
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestImpl.java
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/chrome/android/java/src/org/chromium/chrome/browser/payments/PaymentRequestMetrics.java
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestMetricsTest.java
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestShowTwiceTest.java
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/chrome/android/javatests/src/org/chromium/chrome/browser/payments/PaymentRequestTestCommon.java
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/chrome/browser/payments/android/journey_logger_android.cc
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/chrome/browser/payments/android/journey_logger_android.h
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/components/payments/content/payment_request.cc
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/components/payments/core/journey_logger.cc
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/components/payments/core/journey_logger.h
[modify] https://crrev.com/dfb0b7491cf5b22135241076e5af33d59be918fe/components/payments/core/journey_logger_unittest.cc

Comment 5 by se...@chromium.org, May 29 2017

Status: Fixed (was: Started)
Components: -UI>Browser>Autofill>Payments UI>Browser>Payments

Sign in to add a comment