PaymentRequest Mojo bindings.

mojo/public/cpp/bindings/lib/wtf_string_serialization.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 "mojo/public/cpp/bindings/lib/wtf_string_serialization.h"
 
 #include <string.h>
 
 #include <queue>
 
@@ skipping 69 matching lines @@
     *output = nullptr;
     return;
   }
 
   auto& utf8_adaptors =
       static_cast<WTFStringContextImpl*>(context->wtf_string_context.get())
           ->utf8_adaptors();
 
   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;
 
   mojo::internal::String_Data* result =
       mojo::internal::String_Data::New(adaptor.length(), buf);
   if (result)
     memcpy(result->storage(), adaptor.data(), adaptor.length());
 
   utf8_adaptors.pop();
 
   *output = result;
 }
 
 bool Deserialize_(mojo::internal::String_Data* input,
                   WTF::String* output,
                   mojo::internal::SerializationContext* context) {
   if (input) {
     WTF::String result = WTF::String::fromUTF8(input->storage(), input->size());
     output->swap(result);
   } else if (!output->isNull()) {
     WTF::String result;
     output->swap(result);
   }
   return true;
 }
 
 }  // namespace WTF