Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(102)

Unified Diff: third_party/WebKit/Source/bindings/core/v8/SerializedScriptValueFactory.cpp

Issue 2755383004: Encapsulate optional SerializedScriptValue serialize/deserialize parameters. (Closed)
Patch Set: fuzzer Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/bindings/core/v8/SerializedScriptValueFactory.cpp
diff --git a/third_party/WebKit/Source/bindings/core/v8/SerializedScriptValueFactory.cpp b/third_party/WebKit/Source/bindings/core/v8/SerializedScriptValueFactory.cpp
index 84b662a55bdeffd5a37bf2c8b2c068991de43b82..3a42f8236f4c2a6cec26b1b42a18b4a24175d66e 100644
--- a/third_party/WebKit/Source/bindings/core/v8/SerializedScriptValueFactory.cpp
+++ b/third_party/WebKit/Source/bindings/core/v8/SerializedScriptValueFactory.cpp
@@ -15,24 +15,20 @@ SerializedScriptValueFactory* SerializedScriptValueFactory::m_instance = 0;
PassRefPtr<SerializedScriptValue> SerializedScriptValueFactory::create(
v8::Isolate* isolate,
v8::Local<v8::Value> value,
- Transferables* transferables,
- WebBlobInfoArray* blobInfo,
+ const SerializedScriptValue::SerializeOptions& options,
ExceptionState& exceptionState) {
TRACE_EVENT0("blink", "SerializedScriptValueFactory::create");
- V8ScriptValueSerializer serializer(ScriptState::current(isolate));
- serializer.setBlobInfoArray(blobInfo);
- return serializer.serialize(value, transferables, exceptionState);
+ V8ScriptValueSerializer serializer(ScriptState::current(isolate), options);
+ return serializer.serialize(value, exceptionState);
}
v8::Local<v8::Value> SerializedScriptValueFactory::deserialize(
SerializedScriptValue* value,
v8::Isolate* isolate,
- MessagePortArray* messagePorts,
- const WebBlobInfoArray* blobInfo) {
+ const SerializedScriptValue::DeserializeOptions& options) {
TRACE_EVENT0("blink", "SerializedScriptValueFactory::deserialize");
- V8ScriptValueDeserializer deserializer(ScriptState::current(isolate), value);
- deserializer.setTransferredMessagePorts(messagePorts);
- deserializer.setBlobInfoArray(blobInfo);
+ V8ScriptValueDeserializer deserializer(ScriptState::current(isolate), value,
+ options);
return deserializer.deserialize();
}

Powered by Google App Engine
This is Rietveld 408576698