[Web Payments] Add the shipping method section to the payment sheet

components/payments/content/payment_request.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 "components/payments/content/payment_request.h"
 
 #include <algorithm>
+#include <set>
 #include <unordered_map>
 #include <utility>
 
 #include "base/memory/ptr_util.h"
 #include "components/autofill/core/browser/autofill_data_util.h"
 #include "components/autofill/core/browser/field_types.h"
 #include "components/autofill/core/browser/personal_data_manager.h"
 #include "components/payments/content/payment_details_validation.h"
 #include "components/payments/content/payment_request_web_contents_manager.h"
 #include "components/payments/core/autofill_payment_instrument.h"
@@ skipping 43 matching lines @@
     LOG(ERROR) << error;
     OnConnectionTerminated();
     return;
   }
   client_ = std::move(client);
   details_ = std::move(details);
   options_ = std::move(options);
   PopulateValidatedMethodData(method_data);
   PopulateProfileCache();
   SetDefaultProfileSelections();
+  UpdateSelectedShippingOptionFromDetails();
 }
 
 void PaymentRequest::Show() {
   if (!client_.is_bound() || !binding_.is_bound()) {
     LOG(ERROR) << "Attempted Show(), but binding(s) missing.";
     OnConnectionTerminated();
     return;
   }
   delegate_->ShowDialog(this);
 }
@@ skipping 299 matching lines @@
        selected_contact_profile_
            ->GetInfo(autofill::AutofillType(autofill::PHONE_HOME_WHOLE_NUMBER),
                      app_locale)
            .empty())) {
     return false;
   }
 
   return true;
 }
 
+void PaymentRequest::UpdateSelectedShippingOptionFromDetails() {
+  selected_shipping_option_ = nullptr;
+
+  // As per the spec, the selected shipping option should initially be the last
+  // one in the array that has its selected field set to true.
+  auto selected_shipping_option_it = std::find_if(
+      details()->shipping_options.rbegin(), details()->shipping_options.rend(),
+      [](const payments::mojom::PaymentShippingOptionPtr& element) {
+        return element->selected;
+      });
+  if (selected_shipping_option_it != details()->shipping_options.rend()) {
+    selected_shipping_option_ = selected_shipping_option_it->get();

[Mathieu, 2017/03/09 19:07:03]

I think it's fine to keep the pointer because |details_| is owned by this
PaymentRequest and will outlive it. Maybe you can write that comment in the .h 

[anthonyvd, 2017/03/09 19:43:55]

Done.

+  }
+}
+
 }  // namespace payments