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

components/payments/content/payment_request_web_contents_manager.cc

 // Copyright 2017 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.
 
 #include "components/payments/content/payment_request_web_contents_manager.h"
 
 #include <utility>
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "components/payments/content/payment_request.h"
 #include "components/payments/core/payment_request_delegate.h"
+#include "content/public/browser/navigation_handle.h"
 #include "content/public/browser/web_contents.h"
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(payments::PaymentRequestWebContentsManager);
 
 namespace payments {
 
 PaymentRequestWebContentsManager::~PaymentRequestWebContentsManager() {}
 
 PaymentRequestWebContentsManager*
 PaymentRequestWebContentsManager::GetOrCreateForWebContents(
     content::WebContents* web_contents) {
   DCHECK(web_contents);
   // CreateForWebContents does nothing if the manager instance already exists.
   PaymentRequestWebContentsManager::CreateForWebContents(web_contents);
   return PaymentRequestWebContentsManager::FromWebContents(web_contents);
 }
 
 void PaymentRequestWebContentsManager::CreatePaymentRequest(
     content::WebContents* web_contents,
     std::unique_ptr<PaymentRequestDelegate> delegate,
     mojo::InterfaceRequest<payments::mojom::PaymentRequest> request,
     PaymentRequest::ObserverForTest* observer_for_testing) {
   auto new_request = base::MakeUnique<PaymentRequest>(
       web_contents, std::move(delegate), this, std::move(request),
       observer_for_testing);
   PaymentRequest* request_ptr = new_request.get();
   payment_requests_.insert(std::make_pair(request_ptr, std::move(new_request)));
 }
 
+void PaymentRequestWebContentsManager::DidStartNavigation(
+    content::NavigationHandle* navigation_handle) {
+  for (auto& it : payment_requests_) {

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

PaymentRequest& ?

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

the elements are std::pair<PaymentRequest*, std::unique_ptr<PaymentRequest>> no?
Is there a way to only iterate on the values?

+    // Since the PaymentRequest dialog blocks the content of the page, the user
+    // cannot click on a link to navigate away. Therefore, if the navigation
+    // is initiated in the renderer, it does not come from the user.
+    it.second->NavigatedAway(!navigation_handle->IsRendererInitiated());
+  }
+}
+
 void PaymentRequestWebContentsManager::DestroyRequest(PaymentRequest* request) {
   payment_requests_.erase(request);
 }
 
 PaymentRequestWebContentsManager::PaymentRequestWebContentsManager(
-    content::WebContents* web_contents) {}
+    content::WebContents* web_contents)
+    : content::WebContentsObserver(web_contents) {}
 
 }  // namespace payments