Early return if we find selected shipping option from last.

third_party/WebKit/Source/modules/payments/PaymentRequest.cpp

 // 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 "modules/payments/PaymentRequest.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/JSONValuesForV8.h"
 #include "bindings/core/v8/ScriptPromiseResolver.h"
 #include "bindings/core/v8/ScriptState.h"
@@ skipping 319 matching lines @@
         methodData->append(PaymentRequest::MethodData(pmd.supportedMethods(), stringifiedData));
     }
 }
 
 String getSelectedShippingOption(const PaymentDetails& details)
 {
     String result;
     if (!details.hasShippingOptions())
         return result;
 
-    for (size_t i = 0; i < details.shippingOptions().size(); ++i) {
+    for (int i = details.shippingOptions().size() - 1; i >= 0; --i) {
         if (details.shippingOptions()[i].hasSelected() && details.shippingOptions()[i].selected()) {
             result = details.shippingOptions()[i].id();

[please use gerrit instead, 2016/07/08 13:53:47]

return details.shippingOption()[i].id();

[pals, 2016/07/08 13:58:57]

Done.

+            return result;
         }
     }
 
     return result;
 }
 
 } // namespace
 
 PaymentRequest* PaymentRequest::create(ScriptState* scriptState, const HeapVector<PaymentMethodData>& methodData, const PaymentDetails& details, ExceptionState& exceptionState)
 {
@@ skipping 271 matching lines @@
 {
     m_completeResolver.clear();
     m_showResolver.clear();
     m_abortResolver.clear();
     if (m_clientBinding.is_bound())
         m_clientBinding.Close();
     m_paymentProvider.reset();
 }
 
 } // namespace blink