[Binding] [Refactoring] Move some create() from SerializedScriptValueFactory to SerializedScriptValu

third_party/WebKit/Source/bindings/core/v8/SerializedScriptValueFactory.cpp

 // Copyright 2014 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 "bindings/core/v8/SerializedScriptValueFactory.h"
 
 #include "bindings/core/v8/ExceptionState.h"
 #include "bindings/core/v8/ScriptValueSerializer.h"
 #include "bindings/core/v8/Transferables.h"
 #include "core/dom/DOMArrayBuffer.h"
 #include "core/dom/MessagePort.h"
 #include "core/frame/ImageBitmap.h"
 #include "wtf/ByteOrder.h"
 #include "wtf/text/StringBuffer.h"
 
 namespace blink {
 
 SerializedScriptValueFactory* SerializedScriptValueFactory::m_instance = 0;
 
 PassRefPtr<SerializedScriptValue> SerializedScriptValueFactory::create(v8::Isolate* isolate, v8::Local<v8::Value> value, SerializedScriptValueWriter& writer, Transferables* transferables, WebBlobInfoArray* blobInfo, ExceptionState& exceptionState)
 {
-    RefPtr<SerializedScriptValue> serializedValue = create();
+    RefPtr<SerializedScriptValue> serializedValue = SerializedScriptValue::create();
     ScriptValueSerializer::Status status;
     String errorMessage;
     {
         v8::TryCatch tryCatch(isolate);
         status = doSerialize(value, writer, transferables, blobInfo, serializedValue.get(), tryCatch, errorMessage, isolate);
         if (status == ScriptValueSerializer::JSException) {
             // If there was a JS exception thrown, re-throw it.
             exceptionState.rethrowV8Exception(tryCatch.Exception());
             return serializedValue.release();
         }
@@ skipping 30 matching lines @@
     TrackExceptionState exceptionState;
     return create(isolate, value, nullptr, exceptionState);
 }
 
 PassRefPtr<SerializedScriptValue> SerializedScriptValueFactory::create(v8::Isolate* isolate, const ScriptValue& value, WebBlobInfoArray* blobInfo, ExceptionState& exceptionState)
 {
     ASSERT(isolate->InContext());
     return create(isolate, value.v8Value(), nullptr, blobInfo, exceptionState);
 }
 
-PassRefPtr<SerializedScriptValue> SerializedScriptValueFactory::createFromWire(const String& data)
-{
-    return adoptRef(new SerializedScriptValue(data));
-}
-
-PassRefPtr<SerializedScriptValue> SerializedScriptValueFactory::createFromWireBytes(const char* data, size_t length)
-{
-    // Decode wire data from big endian to host byte order.
-    ASSERT(!(length % sizeof(UChar)));
-    size_t stringLength = length / sizeof(UChar);
-    StringBuffer<UChar> buffer(stringLength);
-    const UChar* src = reinterpret_cast<const UChar*>(data);
-    UChar* dst = buffer.characters();
-    for (size_t i = 0; i < stringLength; i++)
-        dst[i] = ntohs(src[i]);
-
-    return createFromWire(String::adopt(buffer));
-}
-
 PassRefPtr<SerializedScriptValue> SerializedScriptValueFactory::create(const String& data)
 {
-    return create(v8::Isolate::GetCurrent(), data);
-}
-
-PassRefPtr<SerializedScriptValue> SerializedScriptValueFactory::create(v8::Isolate* isolate, const String& data)
-{
-    ASSERT_NOT_REACHED();
     SerializedScriptValueWriter writer;
     writer.writeWebCoreString(data);
     String wireData = writer.takeWireString();
-    return createFromWire(wireData);
-}
-
-PassRefPtr<SerializedScriptValue> SerializedScriptValueFactory::create()
-{
-    return adoptRef(new SerializedScriptValue());
+    return SerializedScriptValue::create(wireData);
 }
 
 void SerializedScriptValueFactory::transferData(SerializedScriptValue* serializedValue, SerializedScriptValueWriter& writer, Transferables* transferables, ExceptionState& exceptionState, v8::Isolate* isolate)
 {
     serializedValue->setData(writer.takeWireString());
     ASSERT(serializedValue->data().impl()->hasOneRef());
     if (!transferables)
         return;
 
     serializedValue->transferImageBitmaps(isolate, transferables->imageBitmaps, exceptionState);
@@ skipping 39 matching lines @@
     SerializedScriptValueReader reader(reinterpret_cast<const uint8_t*>(data.impl()->characters16()), 2 * data.length(), blobInfo, blobDataHandles, ScriptState::current(isolate));
     ScriptValueDeserializer deserializer(reader, messagePorts, arrayBufferContentsArray, imageBitmapContentsArray);
 
     // deserialize() can run arbitrary script (e.g., setters), which could result in |this| being destroyed.
     // Holding a RefPtr ensures we are alive (along with our internal data) throughout the operation.
     return deserializer.deserialize();
 }
 
 
 } // namespace blink