PaymentRequest API.

third_party/WebKit/Source/bindings/core/v8/JSONValuesForV8.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 "platform/JSONValuesForV8.h"
+#include "bindings/core/v8/JSONValuesForV8.h"
+
+#include "bindings/core/v8/ExceptionState.h"
+#include "bindings/core/v8/V8Binding.h"
 
 namespace blink {
 
 static String coreString(v8::Local<v8::String> v8String)
 {
     int length = v8String->Length();
     UChar* buffer;
     String result = String::createUninitialized(length, buffer);
     v8String->Write(reinterpret_cast<uint16_t*>(buffer), 0, length);
     return result;
@@ skipping 60 matching lines @@
             if (!propertyValue)
                 return nullptr;
             jsonObject->setValue(coreString(propertyName), propertyValue);
         }
         return jsonObject;
     }
     ASSERT_NOT_REACHED();
     return nullptr;
 }
 
+v8::Local<v8::Value> fromJSONString(v8::Local<v8::Context> context, const String& stringifiedJSON, ExceptionState& exceptionState)

[haraken, 2016/03/02 00:46:16]

Nit: You can pass in ScriptState rather than v8::Context.

[please use gerrit instead, 2016/03/02 01:24:34]

Done.

+{
+    v8::Isolate* isolate = context->GetIsolate();
+    v8::Local<v8::Value> parsed;
+    v8::TryCatch tryCatch(isolate);

[haraken, 2016/03/02 00:46:16]

Use scriptState->isolate().

[please use gerrit instead, 2016/03/02 01:24:34]

Done.

+    if (!v8Call(v8::JSON::Parse(isolate, v8String(isolate, stringifiedJSON)), parsed, tryCatch)) {
+        if (tryCatch.HasCaught())
+            exceptionState.rethrowV8Exception(tryCatch.Exception());

[haraken, 2016/03/02 00:46:16]

Have you confirmed that this throws an exception aligned with the spec?

[please use gerrit instead, 2016/03/02 01:24:34]

For PaymentResponse, the stringified JSON comes from the browser. The browser
should make sure that it's valid JSON. Therefore, this exception should not
fire. The spec does not mention anything on this topic, but it's not 100%
finalized yet.

+    }
+
+    return parsed;
+}
+
 } // namespace blink