[Payments] Improve the closing of the PR dialog.

chrome/browser/payments/chrome_payment_request_delegate.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 "chrome/browser/payments/chrome_payment_request_delegate.h"
 
 #include "chrome/browser/autofill/personal_data_manager_factory.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser_dialogs.h"
+#include "components/payments/payment_request_dialog.h"
 #include "content/public/browser/web_contents.h"
 
 namespace payments {
 
 ChromePaymentRequestDelegate::ChromePaymentRequestDelegate(
     content::WebContents* web_contents)
-  : web_contents_(web_contents) {}
+    : dialog_(nullptr), web_contents_(web_contents) {}
 
-void ChromePaymentRequestDelegate::ShowPaymentRequestDialog(
-    PaymentRequest* request) {
-  chrome::ShowPaymentRequestDialog(request);
+void ChromePaymentRequestDelegate::ShowDialog(PaymentRequest* request) {

[please use gerrit instead, 2017/01/23 21:18:50]

nit: Would it be a good idea to add "DCHECK_EQ(dialog_, nullptr)"?

[Mathieu, 2017/01/23 21:53:57]

Done.

+  dialog_ = chrome::CreatePaymentRequestDialog(request);
+  dialog_->ShowDialog();
+}
+
+void ChromePaymentRequestDelegate::CloseDialog() {
+  if (dialog_) {

[please use gerrit instead, 2017/01/23 21:18:50]

nit: no {}

[Mathieu, 2017/01/23 21:53:57]

Done.

+    dialog_->CloseDialog();
+  }

[please use gerrit instead, 2017/01/23 21:18:50]

nit: would it be a good idea to add "dialog_ = nullptr"?

[Mathieu, 2017/01/23 21:53:57]

Yes that's not a bad idea. Gets you back into a better state.

 }
 
 autofill::PersonalDataManager*
 ChromePaymentRequestDelegate::GetPersonalDataManager() {
   return autofill::PersonalDataManagerFactory::GetForProfile(
       Profile::FromBrowserContext(web_contents_->GetBrowserContext()));
 }
 
 }  // namespace payments