[PaymentReqeust] Add error message in PaymentDetails. (in blink side)

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 107 matching lines @@
         if (input.hasShippingOptions())
             output->shipping_options = mojo::WTFArray<PaymentShippingOptionPtr>::From(input.shippingOptions());
         else
             output->shipping_options = mojo::WTFArray<PaymentShippingOptionPtr>::New(0);
 
         if (input.hasModifiers())
             output->modifiers = mojo::WTFArray<PaymentDetailsModifierPtr>::From(input.modifiers());
         else
             output->modifiers = mojo::WTFArray<PaymentDetailsModifierPtr>::New(0);
 
+        if (input.hasError())
+            output->error = input.error();
+        else
+            output->error = WTF::String("");

[haraken, 2016/09/20 14:46:53]

WTF::emptyString() ?

[Hwanseung Lee, 2016/09/20 15:08:44]

it is my intention. thank you.

+
         return output;
     }
 };
 
 template <>
 struct TypeConverter<PaymentOptionsPtr, blink::PaymentOptions> {
     static PaymentOptionsPtr Convert(const blink::PaymentOptions& input)
     {
         PaymentOptionsPtr output = PaymentOptions::New();
         output->request_payer_email = input.requestPayerEmail();
@@ skipping 374 matching lines @@
 
     if (!scriptState->domWindow()->frame() || !scriptState->domWindow()->frame()->isMainFrame()) {
         exceptionState.throwSecurityError("Must be in a top-level browsing context");
         return;
     }
 
     validatePaymentDetails(details, exceptionState);
     if (exceptionState.hadException())
         return;
 
+    if (details.hasError() && !details.error().isEmpty()) {
+        exceptionState.throwTypeError("must be Error value  empty");

[please use gerrit instead, 2016/09/20 15:40:50]

"Error value should be empty"

[Hwanseung Lee, 2016/09/20 16:01:09]

Done.

+        return;
+    }
+
     if (m_options.requestShipping())
         m_shippingOption = getSelectedShippingOption(details);
 
     scriptState->domWindow()->frame()->interfaceProvider()->getInterface(mojo::GetProxy(&m_paymentProvider));
     m_paymentProvider.set_connection_error_handler(convertToBaseCallback(WTF::bind(&PaymentRequest::OnError, wrapWeakPersistent(this), mojom::blink::PaymentErrorReason::UNKNOWN)));
     m_paymentProvider->Init(m_clientBinding.CreateInterfacePtrAndBind(), mojo::WTFArray<mojom::blink::PaymentMethodDataPtr>::From(validatedMethodData), mojom::blink::PaymentDetails::From(details), mojom::blink::PaymentOptions::From(m_options));
 }
 
 void PaymentRequest::contextDestroyed()
 {
@@ skipping 164 matching lines @@
     m_completeTimer.stop();
     m_completeResolver.clear();
     m_showResolver.clear();
     m_abortResolver.clear();
     if (m_clientBinding.is_bound())
         m_clientBinding.Close();
     m_paymentProvider.reset();
 }
 
 } // namespace blink