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

Unified Diff: third_party/WebKit/Source/bindings/modules/v8/SerializedScriptValueForModulesFactory.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/modules/v8/SerializedScriptValueForModulesFactory.cpp
diff --git a/third_party/WebKit/Source/bindings/modules/v8/SerializedScriptValueForModulesFactory.cpp b/third_party/WebKit/Source/bindings/modules/v8/SerializedScriptValueForModulesFactory.cpp
index adbf0630fb5501c2d170ef76158d784480b992ff..ce4464d2c221880085898382f46d8eee423d74c0 100644
--- a/third_party/WebKit/Source/bindings/modules/v8/SerializedScriptValueForModulesFactory.cpp
+++ b/third_party/WebKit/Source/bindings/modules/v8/SerializedScriptValueForModulesFactory.cpp
@@ -11,27 +11,24 @@
namespace blink {
PassRefPtr<SerializedScriptValue>
-SerializedScriptValueForModulesFactory::create(v8::Isolate* isolate,
- v8::Local<v8::Value> value,
- Transferables* transferables,
- WebBlobInfoArray* blobInfo,
- ExceptionState& exceptionState) {
+SerializedScriptValueForModulesFactory::create(
+ v8::Isolate* isolate,
+ v8::Local<v8::Value> value,
+ const SerializedScriptValue::SerializeOptions& options,
+ ExceptionState& exceptionState) {
TRACE_EVENT0("blink", "SerializedScriptValueFactory::create");
- V8ScriptValueSerializerForModules serializer(ScriptState::current(isolate));
- serializer.setBlobInfoArray(blobInfo);
- return serializer.serialize(value, transferables, exceptionState);
+ V8ScriptValueSerializerForModules serializer(ScriptState::current(isolate),
+ options);
+ return serializer.serialize(value, exceptionState);
}
v8::Local<v8::Value> SerializedScriptValueForModulesFactory::deserialize(
SerializedScriptValue* value,
v8::Isolate* isolate,
- MessagePortArray* messagePorts,
- const WebBlobInfoArray* blobInfo) {
+ const SerializedScriptValue::DeserializeOptions& options) {
TRACE_EVENT0("blink", "SerializedScriptValueFactory::deserialize");
V8ScriptValueDeserializerForModules deserializer(
- ScriptState::current(isolate), value);
- deserializer.setTransferredMessagePorts(messagePorts);
- deserializer.setBlobInfoArray(blobInfo);
+ ScriptState::current(isolate), value, options);
return deserializer.deserialize();
}

Powered by Google App Engine
This is Rietveld 408576698