[Payments] Only record aborts for nav. in main frame for PR.
Merchant aborts could be recorded when an iframe navigated or if there
were other changes related to the same document. A history pushState
for example.
Since the only way a navigation can close a Payment Request is when the
main frame is navigating to a different document, that's the only
situation in which we should record the abort.
This CL only affects metrics, there should be no user visible change.
Bug: 776474
Change-Id: Ibe8ffc41afbafdf645dc7d970bfc4632ce03e04a
Reviewed-on: https://chromium-review.googlesource.com/728859
Reviewed-by: Mathieu Perreault <[email protected]>
Commit-Queue: Sebastien Seguin-Gagnon <[email protected]>
Cr-Commit-Position: refs/heads/master@{#510491}diff --git a/components/payments/content/payment_request.cc b/components/payments/content/payment_request.cc
index 725dd614..f88555a 100644
--- a/components/payments/content/payment_request.cc
+++ b/components/payments/content/payment_request.cc
@@ -289,7 +289,8 @@
manager_->DestroyRequest(this);
}
-void PaymentRequest::DidStartNavigation(bool is_user_initiated) {
+void PaymentRequest::DidStartMainFrameNavigationToDifferentDocument(
+ bool is_user_initiated) {
RecordFirstAbortReason(is_user_initiated
? JourneyLogger::ABORT_REASON_USER_NAVIGATION
: JourneyLogger::ABORT_REASON_MERCHANT_NAVIGATION);
