PaymentRequest Mojo bindings.

mojo/public/cpp/bindings/lib/wtf_string_serialization.cc

Index: mojo/public/cpp/bindings/lib/wtf_string_serialization.cc
diff --git a/mojo/public/cpp/bindings/lib/wtf_string_serialization.cc b/mojo/public/cpp/bindings/lib/wtf_string_serialization.cc
index d1b749d210c01a709efce47dd5706de2074f8348..d0baaa939159cb89c4f11733d3b1e64e71670049 100644
--- a/mojo/public/cpp/bindings/lib/wtf_string_serialization.cc
+++ b/mojo/public/cpp/bindings/lib/wtf_string_serialization.cc
@@ -88,7 +88,8 @@ void Serialize_(const WTF::String& input,
   DCHECK(!utf8_adaptors.empty());
 #if DCHECK_IS_ON()
   DCHECK_EQ(utf8_adaptors.front().original_input,

[please use gerrit instead, 2016/03/17 19:34:31]

yzshen@: Why would this DCHECK?

For context, here's the JavaScript code that triggers it in my patch:

    var request = new PaymentRequest(
            ["foo"],
            {
                "items": [{"id": "total", "label": "Total", "amount":
{"currencyCode": "USD", "value": "10.00"}}],
                "shippingOptions": [{"id": "self", "label": "Pick up", "amount":
{"currencyCode": "USD", "value": "-5.00"}}]
            },
            {"requestShipping": true},
            {"foo": {"gateway": "bar"}});
    request.show();

The exception stack trace in debug mode:

[1:1:0317/111607:FATAL:wtf_string_serialization.cc(91)] Check failed:
utf8_adaptors.front().original_input == reinterpret_cast<uintptr_t>(&input)
(20812052480624 vs. 140734308029200) string: 'USD'
#0 0x7f6130d1200e base::debug::StackTrace::StackTrace()
#1 0x7f6130d6d43f logging::LogMessage::~LogMessage()
#2 0x7f611cd4296d WTF::Serialize_()
#3 0x7f611cd343e9 blink::mojom::Serialize_()
#4 0x7f611cd34ee1 blink::mojom::Serialize_()
#5 0x7f611cd3ed45 mojo::internal::ArraySerializer<>::SerializeCaller<>::Run()
#6 0x7f611cd3ea06 mojo::internal::ArraySerializer<>::SerializeElements()
#7 0x7f611cd3e86e mojo::internal::ArraySerializationStrategy<>::Serialize<>()
#8 0x7f611cd3e6bf mojo::internal::ArraySerializationImpl<>::Serialize<>()
#9 0x7f611cd381f6 mojo::SerializeArray_<>()
#10 0x7f611cd314d6 blink::mojom::Serialize_()
#11 0x7f611cd30e32 blink::mojom::PaymentRequestProxy::Show()
#12 0x7f611c795815 blink::PaymentRequest::show()
#13 0x7f611ca8f624 blink::PaymentRequestV8Internal::showMethod()
#14 0x7f611ca8f585 blink::PaymentRequestV8Internal::showMethodCallback()
#15 0x7f612457db7b v8::internal::FunctionCallbackArguments::Call()
#16 0x7f61245cb651 v8::internal::(anonymous namespace)::HandleApiCallHelper<>()
#17 0x7f6124600d27 v8::internal::Builtin_Impl_HandleApiCall()
#18 0x7f61245dc219 v8::internal::Builtin_HandleApiCall()

[yzshen1, 2016/03/17 21:41:17]

I will look into it. Would you please tell me the repo steps (what patch to
apply, what target I should build and run)?

-            reinterpret_cast<uintptr_t>(&input));
+            reinterpret_cast<uintptr_t>(&input))
+      << " string:'" << input.utf8().data() << "'";
 #endif
 
   const WTF::StringUTF8Adaptor& adaptor = utf8_adaptors.front().utf8_adaptor;