Blink-compatible serialization of Boolean, Number and String objects.

test/unittests/value-serializer-unittest.cc

 // Copyright 2016 the V8 project 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 "src/value-serializer.h"
 
 #include <algorithm>
 #include <string>
 
 #include "include/v8.h"
@@ skipping 65 matching lines @@
 
   template <typename MessageFunctor>
   void InvalidEncodeTest(const char* source, const MessageFunctor& functor) {
     Context::Scope scope(serialization_context());
     TryCatch try_catch(isolate());
     Local<Value> input_value = EvaluateScriptForInput(source);
     ASSERT_TRUE(DoEncode(input_value).IsNothing());
     functor(try_catch.Message());
   }
 
+  void InvalidEncodeTest(const char* source) {
+    InvalidEncodeTest(source, [](Local<Message>) {});
+  }
+
   template <typename OutputFunctor>
   void DecodeTest(const std::vector<uint8_t>& data,
                   const OutputFunctor& output_functor) {
     Context::Scope scope(deserialization_context());
     TryCatch try_catch(isolate());
     // TODO(jbroman): Use the public API once it exists.
     i::Isolate* internal_isolate = reinterpret_cast<i::Isolate*>(isolate());
     i::HandleScope handle_scope(internal_isolate);
     i::ValueDeserializer deserializer(
         internal_isolate,
@@ skipping 1023 matching lines @@
   DecodeTest(
       {0xff, 0x09, 0x3f, 0x00, 0x6f, 0x3f, 0x01, 0x53, 0x01, 0x61, 0x3f,
        0x01, 0x44, 0x00, 0x20, 0x39, 0x50, 0x37, 0x6a, 0x75, 0x42, 0x3f,
        0x02, 0x53, 0x01, 0x62, 0x3f, 0x02, 0x5e, 0x01, 0x7b, 0x02},
       [this](Local<Value> value) {
         EXPECT_TRUE(EvaluateScriptForResultBool("result.a instanceof Date"));
         EXPECT_TRUE(EvaluateScriptForResultBool("result.a === result.b"));
       });
 }
 
+TEST_F(ValueSerializerTest, RoundTripValueObjects) {
+  RoundTripTest("new Boolean(true)", [this](Local<Value> value) {
+    EXPECT_TRUE(EvaluateScriptForResultBool(
+        "Object.getPrototypeOf(result) === Boolean.prototype"));
+    EXPECT_TRUE(EvaluateScriptForResultBool("result.valueOf() === true"));
+  });
+  RoundTripTest("new Boolean(false)", [this](Local<Value> value) {
+    EXPECT_TRUE(EvaluateScriptForResultBool(
+        "Object.getPrototypeOf(result) === Boolean.prototype"));
+    EXPECT_TRUE(EvaluateScriptForResultBool("result.valueOf() === false"));
+  });
+  RoundTripTest(
+      "({ a: new Boolean(true), get b() { return this.a; }})",
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a instanceof Boolean"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a === result.b"));
+      });
+  RoundTripTest("new Number(-42)", [this](Local<Value> value) {
+    EXPECT_TRUE(EvaluateScriptForResultBool(
+        "Object.getPrototypeOf(result) === Number.prototype"));
+    EXPECT_TRUE(EvaluateScriptForResultBool("result.valueOf() === -42"));
+  });
+  RoundTripTest("new Number(NaN)", [this](Local<Value> value) {
+    EXPECT_TRUE(EvaluateScriptForResultBool(
+        "Object.getPrototypeOf(result) === Number.prototype"));
+    EXPECT_TRUE(EvaluateScriptForResultBool("Number.isNaN(result.valueOf())"));
+  });
+  RoundTripTest(
+      "({ a: new Number(6), get b() { return this.a; }})",
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a instanceof Number"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a === result.b"));
+      });
+  RoundTripTest("new String('Qu\\xe9bec')", [this](Local<Value> value) {
+    EXPECT_TRUE(EvaluateScriptForResultBool(
+        "Object.getPrototypeOf(result) === String.prototype"));
+    EXPECT_TRUE(
+        EvaluateScriptForResultBool("result.valueOf() === 'Qu\\xe9bec'"));
+    EXPECT_TRUE(EvaluateScriptForResultBool("result.length === 6"));
+  });
+  RoundTripTest("new String('\\ud83d\\udc4a')", [this](Local<Value> value) {
+    EXPECT_TRUE(EvaluateScriptForResultBool(
+        "Object.getPrototypeOf(result) === String.prototype"));
+    EXPECT_TRUE(
+        EvaluateScriptForResultBool("result.valueOf() === '\\ud83d\\udc4a'"));
+    EXPECT_TRUE(EvaluateScriptForResultBool("result.length === 2"));
+  });
+  RoundTripTest(
+      "({ a: new String(), get b() { return this.a; }})",
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a instanceof String"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a === result.b"));
+      });
+}
+
+TEST_F(ValueSerializerTest, RejectsOtherValueObjects) {
+  // This is a roundabout way of getting an instance of Symbol.
+  InvalidEncodeTest("Object.valueOf.apply(Symbol())");

[Jakob Kummerow, 2016/08/22 17:42:22]

Why not just "Symbol()"?

[jbroman, 2016/08/22 19:02:33]

Symbol() isn't an object; it's a value of type 'symbol'. I wanted an object
wrapper for a symbol (i.e., an object that is an instance of Symbol). Unlike
Boolean, Number and String, the constructor cannot be directly used.
Conceptually, this is "new Symbol(Symbol())".

[Jakob Kummerow, 2016/08/23 09:43:14]

Acknowledged.

+}
+
+TEST_F(ValueSerializerTest, DecodeValueObjects) {
+  DecodeTest(
+      {0xff, 0x09, 0x3f, 0x00, 0x79, 0x00},
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool(
+            "Object.getPrototypeOf(result) === Boolean.prototype"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.valueOf() === true"));
+      });
+  DecodeTest(
+      {0xff, 0x09, 0x3f, 0x00, 0x78, 0x00},
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool(
+            "Object.getPrototypeOf(result) === Boolean.prototype"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.valueOf() === false"));
+      });
+  DecodeTest(
+      {0xff, 0x09, 0x3f, 0x00, 0x6f, 0x3f, 0x01, 0x53, 0x01, 0x61, 0x3f, 0x01,
+       0x79, 0x3f, 0x02, 0x53, 0x01, 0x62, 0x3f, 0x02, 0x5e, 0x01, 0x7b, 0x02},
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a instanceof Boolean"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a === result.b"));
+      });
+  DecodeTest(
+      {0xff, 0x09, 0x3f, 0x00, 0x6e, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x45,
+       0xc0, 0x00},
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool(
+            "Object.getPrototypeOf(result) === Number.prototype"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.valueOf() === -42"));
+      });
+  DecodeTest({0xff, 0x09, 0x3f, 0x00, 0x6e, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00,
+              0xf8, 0x7f, 0x00},
+             [this](Local<Value> value) {
+               EXPECT_TRUE(EvaluateScriptForResultBool(
+                   "Object.getPrototypeOf(result) === Number.prototype"));
+               EXPECT_TRUE(EvaluateScriptForResultBool(
+                   "Number.isNaN(result.valueOf())"));
+             });
+  DecodeTest(
+      {0xff, 0x09, 0x3f, 0x00, 0x6f, 0x3f, 0x01, 0x53, 0x01, 0x61, 0x3f,
+       0x01, 0x6e, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x18, 0x40, 0x3f,
+       0x02, 0x53, 0x01, 0x62, 0x3f, 0x02, 0x5e, 0x01, 0x7b, 0x02},
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a instanceof Number"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a === result.b"));
+      });
+  DecodeTest({0xff, 0x09, 0x3f, 0x00, 0x73, 0x07, 0x51, 0x75, 0xc3, 0xa9, 0x62,
+              0x65, 0x63, 0x00},
+             [this](Local<Value> value) {
+               EXPECT_TRUE(EvaluateScriptForResultBool(
+                   "Object.getPrototypeOf(result) === String.prototype"));
+               EXPECT_TRUE(EvaluateScriptForResultBool(
+                   "result.valueOf() === 'Qu\\xe9bec'"));
+               EXPECT_TRUE(EvaluateScriptForResultBool("result.length === 6"));
+             });
+  DecodeTest({0xff, 0x09, 0x3f, 0x00, 0x73, 0x04, 0xf0, 0x9f, 0x91, 0x8a},
+             [this](Local<Value> value) {
+               EXPECT_TRUE(EvaluateScriptForResultBool(
+                   "Object.getPrototypeOf(result) === String.prototype"));
+               EXPECT_TRUE(EvaluateScriptForResultBool(
+                   "result.valueOf() === '\\ud83d\\udc4a'"));
+               EXPECT_TRUE(EvaluateScriptForResultBool("result.length === 2"));
+             });
+  DecodeTest(
+      {0xff, 0x09, 0x3f, 0x00, 0x6f, 0x3f, 0x01, 0x53, 0x01,
+       0x61, 0x3f, 0x01, 0x73, 0x00, 0x3f, 0x02, 0x53, 0x01,
+       0x62, 0x3f, 0x02, 0x5e, 0x01, 0x7b, 0x02, 0x00},
+      [this](Local<Value> value) {
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a instanceof String"));
+        EXPECT_TRUE(EvaluateScriptForResultBool("result.a === result.b"));
+      });
+}
+
 }  // namespace
 }  // namespace v8