[Payments] Record navigations that close the Payment Request as aborts.

components/payments/content/payment_request.h

Index: components/payments/content/payment_request.h
diff --git a/components/payments/content/payment_request.h b/components/payments/content/payment_request.h
index c831cb916e8f89f96dfae69fddd7527ca1e4966f..0badb9d1b1cad6715a9b311e4f4c0f717fb22857 100644
--- a/components/payments/content/payment_request.h
+++ b/components/payments/content/payment_request.h
@@ -75,6 +75,10 @@ class PaymentRequest : public mojom::PaymentRequest,
   // OnConnectionTerminated).
   void UserCancelled();
 
+  // Called when there is a navigation out of a page with a PaymentRequest

[Mathieu, 2017/05/09 18:12:19]

// Called when the frame attached to this PaymentRequest is navigating away, but
before the PaymentRequest is destroyed.

BTW does this include reloads and tab closing?

[sebsg, 2017/05/10 14:58:48]

Reload triggers it but not a tab closing. However, tabclosing already registers
as a User Abort. I'll add a test.

+  // dialog.
+  void NavigatedAway(bool is_user_initiated);

[Mathieu, 2017/05/09 18:12:19]

DidStartNavigation(bool is_user_initiated) is a more descriptive name.

[sebsg, 2017/05/10 14:58:48]

Done.

+
   // As a result of a browser-side error or renderer-initiated mojo channel
   // closure (e.g. there was an error on the renderer side, or payment was
   // successful), this method is called. It is responsible for cleaning up,