[Web Payments] Use shippingType to choose the Payment Sheet shipping strings.

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"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/ui/views/payments/payment_request_sheet_controller.h"
 #include "components/autofill/core/browser/autofill_data_util.h"
 #include "components/autofill/core/browser/autofill_profile.h"
 #include "components/autofill/core/browser/autofill_type.h"
 #include "components/autofill/core/browser/credit_card.h"
 #include "components/autofill/core/browser/field_types.h"
+#include "components/strings/grit/components_strings.h"
 #include "third_party/skia/include/core/SkColor.h"
+#include "ui/base/l10n/l10n_util.h"
 #include "ui/base/resource/resource_bundle.h"
 #include "ui/gfx/canvas.h"
 #include "ui/gfx/font_list.h"
 #include "ui/gfx/geometry/insets.h"
 #include "ui/gfx/geometry/point_f.h"
 #include "ui/gfx/paint_vector_icon.h"
 #include "ui/vector_icons/vector_icons.h"
 #include "ui/views/border.h"
 #include "ui/views/bubble/bubble_frame_view.h"
 #include "ui/views/controls/button/button.h"
@@ skipping 210 matching lines @@
 
 std::unique_ptr<views::Label> CreateBoldLabel(const base::string16& text) {
   std::unique_ptr<views::Label> label = base::MakeUnique<views::Label>(text);
 
   label->SetFontList(
       label->font_list().DeriveWithWeight(gfx::Font::Weight::BOLD));
 
   return label;
 }
 
+base::string16 GetShippingAddressSectionString(
+    const payments::mojom::PaymentShippingType& shipping_type) {
+  if (shipping_type == payments::mojom::PaymentShippingType::DELIVERY)

[Mathieu, 2017/03/10 20:47:29]

Could we have a switch(shipping_type) with three cases, which would complain if
there is ever a new enum value added?

[anthonyvd, 2017/03/10 20:58:53]

Good point, I set it up like this when I was confused and thought the value
would be a string (since that's what javascript passes).

Done.

+    return l10n_util::GetStringUTF16(IDS_PAYMENTS_DELIVERY_ADDRESS_LABEL);
+  if (shipping_type == payments::mojom::PaymentShippingType::PICKUP)
+    return l10n_util::GetStringUTF16(IDS_PAYMENTS_PICKUP_ADDRESS_LABEL);
+
+  return l10n_util::GetStringUTF16(IDS_PAYMENTS_SHIPPING_ADDRESS_LABEL);
+}
+
+base::string16 GetShippingOptionSectionString(
+    const payments::mojom::PaymentShippingType& shipping_type) {
+  if (shipping_type == payments::mojom::PaymentShippingType::DELIVERY)
+    return l10n_util::GetStringUTF16(IDS_PAYMENTS_DELIVERY_OPTION_LABEL);
+  if (shipping_type == payments::mojom::PaymentShippingType::PICKUP)
+    return l10n_util::GetStringUTF16(IDS_PAYMENTS_PICKUP_OPTION_LABEL);
+
+  return l10n_util::GetStringUTF16(IDS_PAYMENTS_SHIPPING_OPTION_LABEL);
+}
+
 }  // namespace payments