[Payment Request] Change the lifetime management of PaymentRequestImpl

components/payments/payment_request_delegate.h

Index: components/payments/payment_request_delegate.h
diff --git a/components/payments/payment_request_delegate.h b/components/payments/payment_request_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..657f3551ac30d433ce3c5352531e9a08d65fa422
--- /dev/null
+++ b/components/payments/payment_request_delegate.h
@@ -0,0 +1,19 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.

[please use gerrit instead, 2017/01/06 21:49:09]

2017

[Mathieu, 2017/01/07 05:03:07]

Done.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef COMPONENTS_PAYMENTS_PAYMENT_REQUEST_DELEGATE_H_
+#define COMPONENTS_PAYMENTS_PAYMENT_REQUEST_DELEGATE_H_
+
+namespace payments {
+
+class PaymentRequestImpl;
+
+class PaymentRequestDelegate {
+ public:
+  virtual void ShowPaymentRequestDialog(PaymentRequestImpl* impl) = 0;

[please use gerrit instead, 2017/01/06 21:49:09]

Need a virtual destructor with empty body:

"To make sure all implementations of the interface can be destroyed correctly,
the interface must also declare a virtual destructor (in an exception to the
first rule, this should not be pure). See Stroustrup, The C++ Programming
Language, 3rd edition, section 12.4 for details."

https://google.github.io/styleguide/cppguide.html#Interfaces

[Mathieu, 2017/01/07 05:03:07]

Done.

+};
+
+}  // namespace payments
+
+#endif  // COMPONENTS_PAYMENTS_PAYMENT_REQUEST_DELEGATE_H_