Use PaymentRequestItemList for Shipping screen

chrome/browser/ui/views/payments/payment_request_views_util.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_views_util.h"
 
 #include <vector>
 
 #include "base/macros.h"
 #include "base/memory/ptr_util.h"
@@ skipping 197 matching lines @@
     values.push_back(name_value);
   if (!phone_value.empty())
     values.push_back(phone_value);
   if (!email_value.empty())
     values.push_back(email_value);
 
   return base::MakeUnique<views::StyledLabel>(
       base::JoinString(values, base::ASCIIToUTF16("\n")), nullptr);
 }
 
-// Creates a views::Border object that can paint the gray horizontal ruler used

[anthonyvd, 2017/03/01 20:15:34]

This comment was on both this and the declaration in the .h, right?

[tmartino, 2017/03/01 22:48:14]

Yes. They were identical, I think it was an artifact of copy-pasting the
declaration into the cc file.

-// as a separator between items in the Payment Request dialog.
 std::unique_ptr<views::Border> CreatePaymentRequestRowBorder() {
   return views::CreateBorderPainter(
       base::MakeUnique<PaymentRequestRowBorderPainter>(),
       gfx::Insets());
 }
 
 }  // namespace payments