[Payments] base::string16 -> std::string in PaymentMethodData

ios/chrome/browser/payments/payment_request.mm

Index: ios/chrome/browser/payments/payment_request.mm
diff --git a/ios/chrome/browser/payments/payment_request.mm b/ios/chrome/browser/payments/payment_request.mm
index aa3d512c72a444114daa822eac42171f24f16a00..4f2ce052fc68e967bdf2c7cd603fea3b7a485bd2 100644
--- a/ios/chrome/browser/payments/payment_request.mm
+++ b/ios/chrome/browser/payments/payment_request.mm
@@ -77,12 +77,10 @@ void PaymentRequest::PopulateProfileCache() {
 
 void PaymentRequest::PopulateCreditCardCache() {
   for (const auto& method_data : web_payment_request_.method_data) {
-    for (const auto& supported_method : method_data.supported_methods) {
+    for (const std::string& supported_method : method_data.supported_methods) {
       // Reject non-ASCII supported methods.
-      if (base::IsStringASCII(supported_method)) {
-        supported_card_networks_.push_back(
-            base::UTF16ToASCII(supported_method));
-      }
+      if (base::IsStringASCII(supported_method))

[Moe, 2017/04/05 13:54:08]

should we have this check in payment_method_data.cc instead?

[Mathieu, 2017/04/05 13:59:18]

Yes sounds like a good idea! Also made the change to supported_networks and
types

+        supported_card_networks_.push_back(supported_method);
     }
   }