Revert of Internalize strings in-place

src/value-serializer.cc

Index: src/value-serializer.cc
diff --git a/src/value-serializer.cc b/src/value-serializer.cc
index 90cb9c92af1cf8f840bcaa57e21de5b7b2e94922..0e3a6a6e1e04346a71f287364bf6408238f63d73 100644
--- a/src/value-serializer.cc
+++ b/src/value-serializer.cc
@@ -1135,9 +1135,8 @@
   if (!ReadVarint<uint32_t>().To(&utf8_length) ||
       utf8_length >
           static_cast<uint32_t>(std::numeric_limits<int32_t>::max()) ||
-      !ReadRawBytes(utf8_length).To(&utf8_bytes)) {
+      !ReadRawBytes(utf8_length).To(&utf8_bytes))
     return MaybeHandle<String>();
-  }
   return isolate_->factory()->NewStringFromUtf8(
       Vector<const char>::cast(utf8_bytes), pretenure_);
 }
@@ -1148,20 +1147,16 @@
   if (!ReadVarint<uint32_t>().To(&byte_length) ||
       byte_length >
           static_cast<uint32_t>(std::numeric_limits<int32_t>::max()) ||
-      byte_length % sizeof(uc16) != 0 ||
-      !ReadRawBytes(byte_length).To(&bytes)) {
+      byte_length % sizeof(uc16) != 0 || !ReadRawBytes(byte_length).To(&bytes))
     return MaybeHandle<String>();
-  }
 
   // Allocate an uninitialized string so that we can do a raw memcpy into the
   // string on the heap (regardless of alignment).
-  if (byte_length == 0) return isolate_->factory()->empty_string();
   Handle<SeqTwoByteString> string;
   if (!isolate_->factory()
            ->NewRawTwoByteString(byte_length / sizeof(uc16), pretenure_)
-           .ToHandle(&string)) {
+           .ToHandle(&string))
     return MaybeHandle<String>();
-  }
 
   // Copy the bytes directly into the new string.
   // Warning: this uses host endianness.