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

components/payments/content/payment_request.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef COMPONENTS_PAYMENTS_CONTENT_PAYMENT_REQUEST_H_
 #define COMPONENTS_PAYMENTS_CONTENT_PAYMENT_REQUEST_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 57 matching lines @@
   // PaymentRequestState::Delegate:
   void OnPaymentResponseAvailable(mojom::PaymentResponsePtr response) override;
   void OnShippingOptionIdSelected(std::string shipping_option_id) override;
   void OnShippingAddressSelected(mojom::PaymentAddressPtr address) override;
 
   // Called when the user explicitely cancelled the flow. Will send a message
   // to the renderer which will indirectly destroy this object (through
   // 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,
   // such as possibly closing the dialog.
   void OnConnectionTerminated();
 
   // Called when the user clicks on the "Pay" button.
   void Pay();
 
   content::WebContents* web_contents() { return web_contents_; }
@@ skipping 16 matching lines @@
   ObserverForTest* observer_for_testing_;
 
   JourneyLogger journey_logger_;
 
   DISALLOW_COPY_AND_ASSIGN(PaymentRequest);
 };
 
 }  // namespace payments
 
 #endif  // COMPONENTS_PAYMENTS_CONTENT_PAYMENT_REQUEST_H_