[WebPayments] Adding something resembling Shipping Address list

chrome/browser/ui/views/payments/payment_request_dialog.cc

 // 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.
 
 #include "chrome/browser/ui/views/payments/payment_request_dialog.h"
 
 #include <utility>
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "chrome/browser/ui/views/payments/order_summary_view_controller.h"
 #include "chrome/browser/ui/views/payments/payment_method_view_controller.h"
 #include "chrome/browser/ui/views/payments/payment_sheet_view_controller.h"
+#include "chrome/browser/ui/views/payments/shipping_list_view_controller.h"
 #include "chrome/grit/generated_resources.h"
 #include "components/constrained_window/constrained_window_views.h"
 #include "components/payments/payment_request.h"
 #include "content/public/browser/browser_thread.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/views/layout/fill_layout.h"
 
 namespace chrome {
 
 void ShowPaymentRequestDialog(payments::PaymentRequest* request) {
@@ skipping 67 matching lines @@
 void PaymentRequestDialog::GoBack() {
   view_stack_.Pop();
 }
 
 void PaymentRequestDialog::ShowOrderSummary() {
   view_stack_.Push(CreateViewAndInstallController<OrderSummaryViewController>(
                        &controller_map_, request_, this),
                    true);
 }
 
+void PaymentRequestDialog::ShowShippingListSheet() {
+  view_stack_.Push(CreateViewAndInstallController<ShippingListViewController>(
+                       &controller_map_, request_, this),
+                   true);

[please use gerrit instead, 2017/01/20 19:32:53]

Please add a comment about the meaning of this "true" parameter.

[tmartino, 2017/01/25 00:08:41]

Done here and for the other calls to Push in this class.

+}
+
 void PaymentRequestDialog::ShowPaymentMethodSheet() {
     view_stack_.Push(
         CreateViewAndInstallController<PaymentMethodViewController>(
             &controller_map_, request_, this),
         true);
 }
 
 void PaymentRequestDialog::CloseDialog() {
   GetWidget()->Close();
 }
@@ skipping 13 matching lines @@
   // When a view that is associated with a controller is removed from this
   // view's descendants, dispose of the controller.
   if (!details.is_add &&
       controller_map_.find(details.child) != controller_map_.end()) {
     DCHECK(!details.move_view);
     controller_map_.erase(details.child);
   }
 }
 
 }  // namespace payments