[WebPayments] Adding Shipping Address and Contact Info display to order summary

chrome/browser/ui/views/payments/payment_request_address_util.h

Index: chrome/browser/ui/views/payments/payment_request_address_util.h
diff --git a/chrome/browser/ui/views/payments/payment_request_address_util.h b/chrome/browser/ui/views/payments/payment_request_address_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..cfefc5d1ac2d84f3fe9c5d43e78cffb776615e6c
--- /dev/null
+++ b/chrome/browser/ui/views/payments/payment_request_address_util.h
@@ -0,0 +1,35 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.

[anthonyvd, 2017/01/12 15:42:11]

Can we move the contents of this file in payment_request_views_util? It's meant
to contain this sort of functions that can be used in multiple places to
create/set up UI pieces. We could also rename it if the name doesn't convey this
well :)

[tmartino, 2017/01/18 17:28:24]

Done

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_VIEWS_PAYMENTS_PAYMENT_REQUEST_ADDRESS_UTIL_H_
+#define CHROME_BROWSER_UI_VIEWS_PAYMENTS_PAYMENT_REQUEST_ADDRESS_UTIL_H_
+
+#include <memory>
+#include <string>
+
+#include "components/autofill/core/browser/autofill_profile.h"
+
+namespace views {
+class View;
+}
+
+namespace payments {
+
+enum class AddressFormatType {
+  SHIPPING_SUMMARY,
+  SHIPPING_DETAILED,
+  CONTACT_SUMMARY,
+  CONTACT_DETAILED
+};
+
+std::unique_ptr<autofill::AutofillProfile> GetDummyProfile();
+
+std::unique_ptr<views::View> GetPaymentRequestAddressLabel(
+    AddressFormatType type,

[anthonyvd, 2017/01/12 15:42:11]

What's this parameter for (and same question for the AddressFormatType enum)? 

[tmartino, 2017/01/18 17:28:24]

Changed name + documented

+    std::string locale,
+    autofill::AutofillProfile* profile);
+
+}  // namespace payments
+
+#endif  // CHROME_BROWSER_UI_VIEWS_PAYMENTS_PAYMENT_REQUEST_ADDRESS_UTIL_H_