Add support for lists to the C message reader

runtime/vm/snapshot_test.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "platform/assert.h"
 #include "vm/bigint_operations.h"
 #include "vm/class_finalizer.h"
 #include "vm/dart_api_impl.h"
 #include "vm/dart_api_state.h"
 #include "vm/snapshot.h"
@@ skipping 45 matching lines @@
   // Read object back from the snapshot.
   SnapshotReader reader(snapshot, Isolate::Current());
   const Object& serialized_object = Object::Handle(reader.ReadObject());
   EXPECT(Equals(null_object, serialized_object));
 
   // Read object back from the snapshot into a C structure.
   CMessageReader mreader(buffer + Snapshot::kHeaderSize,
                         writer.BytesWritten(),
                         &allocator);
   Dart_CObject* cobject = mreader.ReadObject();
+  EXPECT_NOTNULL(cobject);
   EXPECT_EQ(Dart_CObject::kNull, cobject->type);
   free(cobject);
 }
 
 
 TEST_CASE(SerializeSmi1) {
   // Write snapshot with object content.
   uint8_t* buffer;
   SnapshotWriter writer(Snapshot::kMessage, &buffer, &allocator);
   const Smi& smi = Smi::Handle(Smi::New(124));
   writer.WriteObject(smi.raw());
   writer.FinalizeBuffer();
 
   // Create a snapshot object using the buffer.
   const Snapshot* snapshot = Snapshot::SetupFromBuffer(buffer);
 
   // Read object back from the snapshot.
   SnapshotReader reader(snapshot, Isolate::Current());
   const Object& serialized_object = Object::Handle(reader.ReadObject());
   EXPECT(Equals(smi, serialized_object));
 
   // Read object back from the snapshot into a C structure.
   CMessageReader mreader(buffer + Snapshot::kHeaderSize,
                          writer.BytesWritten(),
                          &allocator);
   Dart_CObject* cobject = mreader.ReadObject();
+  EXPECT_NOTNULL(cobject);
   EXPECT_EQ(Dart_CObject::kInt32, cobject->type);
   EXPECT_EQ(smi.Value(), cobject->value.as_int32);
   free(cobject);
 }
 
 
 TEST_CASE(SerializeSmi2) {
   // Write snapshot with object content.
   uint8_t* buffer;
   SnapshotWriter writer(Snapshot::kMessage, &buffer, &allocator);
   const Smi& smi = Smi::Handle(Smi::New(-1));
   writer.WriteObject(smi.raw());
   writer.FinalizeBuffer();
 
   // Create a snapshot object using the buffer.
   const Snapshot* snapshot = Snapshot::SetupFromBuffer(buffer);
 
   // Read object back from the snapshot.
   SnapshotReader reader(snapshot, Isolate::Current());
   const Object& serialized_object = Object::Handle(reader.ReadObject());
   EXPECT(Equals(smi, serialized_object));
 
   // Read object back from the snapshot into a C structure.
   CMessageReader mreader(buffer + Snapshot::kHeaderSize,
                          writer.BytesWritten(),
                          &allocator);
   Dart_CObject* cobject = mreader.ReadObject();
+  EXPECT_NOTNULL(cobject);
   EXPECT_EQ(Dart_CObject::kInt32, cobject->type);
   EXPECT_EQ(smi.Value(), cobject->value.as_int32);
   free(cobject);
 }
 
 
 TEST_CASE(SerializeDouble) {
   // Write snapshot with object content.
   uint8_t* buffer;
   SnapshotWriter writer(Snapshot::kMessage, &buffer, &allocator);
   const Double& dbl = Double::Handle(Double::New(101.29));
   writer.WriteObject(dbl.raw());
   writer.FinalizeBuffer();
 
   // Create a snapshot object using the buffer.
   const Snapshot* snapshot = Snapshot::SetupFromBuffer(buffer);
 
   // Read object back from the snapshot.
   SnapshotReader reader(snapshot, Isolate::Current());
   const Object& serialized_object = Object::Handle(reader.ReadObject());
   EXPECT(Equals(dbl, serialized_object));
 
   // Read object back from the snapshot into a C structure.
   CMessageReader mreader(buffer + Snapshot::kHeaderSize,
                          writer.BytesWritten(),
                          &allocator);
   Dart_CObject* cobject = mreader.ReadObject();
+  EXPECT_NOTNULL(cobject);
   EXPECT_EQ(Dart_CObject::kDouble, cobject->type);
   EXPECT_EQ(dbl.value(), cobject->value.as_double);
   free(cobject);
 }
 
 
 TEST_CASE(SerializeBool) {
   // Write snapshot with object content.
   uint8_t* buffer;
   SnapshotWriter writer(Snapshot::kMessage, &buffer, &allocator);
   const Bool& bool1 = Bool::Handle(Bool::True());
   const Bool& bool2 = Bool::Handle(Bool::False());
   writer.WriteObject(bool1.raw());
   writer.WriteObject(bool2.raw());
   writer.FinalizeBuffer();
 
   // Create a snapshot object using the buffer.
   const Snapshot* snapshot = Snapshot::SetupFromBuffer(buffer);
 
   // Read object back from the snapshot.
   SnapshotReader reader(snapshot, Isolate::Current());
   EXPECT(Bool::True() == reader.ReadObject());
   EXPECT(Bool::False() == reader.ReadObject());
 
   // Read object back from the snapshot into a C structure.
   CMessageReader mreader(buffer + Snapshot::kHeaderSize,
                          writer.BytesWritten(),
                          &allocator);
   Dart_CObject* cobject1 = mreader.ReadObject();
+  EXPECT_NOTNULL(cobject1);
   EXPECT_EQ(Dart_CObject::kBool, cobject1->type);
   EXPECT_EQ(true, cobject1->value.as_bool);
   Dart_CObject* cobject2 = mreader.ReadObject();
+  EXPECT_NOTNULL(cobject2);
   EXPECT_EQ(Dart_CObject::kBool, cobject2->type);
   EXPECT_EQ(false, cobject2->value.as_bool);
   free(cobject1);
   free(cobject2);
 }
 
 
 TEST_CASE(SerializeBigint) {
   // Write snapshot with object content.
   uint8_t* buffer;
@@ skipping 469 matching lines @@
   for (int i = 0; i < kArrayLength; i++) {
     Dart_CObject* element = value->value.as_array.values[i];
     EXPECT_EQ(Dart_CObject::kInt32, element->type);
     EXPECT_EQ(i + 1, element->value.as_int32);
     free(element);
   }
   free(value);
 }
 
 
+// Helper function to call a top level Dart function, serialize the
+// result and deserialize the result into a Dart_CObject structure.
+static Dart_CObject* GetDeserializedDartObject(Dart_Handle lib,
+                                               const char* dart_function) {
+  Dart_Handle result;
+  result = Dart_InvokeStatic(lib,
+                             Dart_NewString(""),
+                             Dart_NewString(dart_function),
+                             0,
+                             NULL);
+  EXPECT_VALID(result);
+
+  // Serialize the list into a message.
+  uint8_t* buffer;
+  SnapshotWriter writer(Snapshot::kMessage, &buffer, &allocator);
+  const Object& list = Object::Handle(Api::UnwrapHandle(result));
+  writer.WriteObject(list.raw());
+  writer.FinalizeBuffer();
+
+  // Read object back from the snapshot into a C structure.
+  CMessageReader reader(buffer + Snapshot::kHeaderSize,
+                        writer.BytesWritten(),
+                        &allocator);
+  Dart_CObject* value = reader.ReadObject();
+  free(buffer);
+  return value;
+}
+
+
 UNIT_TEST_CASE(DartGeneratedMessages) {
-  const int kArrayLength = 10;
   static const char* kCustomIsolateScriptChars =
-    "getSmi() {\n"
-    "  return 42;\n"
-    "}\n"
-    "getString() {\n"
+      "getSmi() {\n"
+      "  return 42;\n"
+      "}\n"
+      "getString() {\n"
       "  return \"Hello, world!\";\n"
-    "}\n"
-    "getList() {\n"
-    "  return [1,2,3,4,5,6,7,8,9,10];\n"
-    "}\n";
+      "}\n"
+      "getList() {\n"
+      "  return new List(kArrayLength);\n"
+      "}\n";
 
   TestCase::CreateTestIsolate();
   Isolate* isolate = Isolate::Current();
   EXPECT(isolate != NULL);
   Dart_EnterScope();
 
   Dart_Handle lib = TestCase::LoadTestScript(kCustomIsolateScriptChars,
                                              NULL);
   EXPECT_VALID(lib);
   Dart_Handle smi_result;
   smi_result = Dart_InvokeStatic(lib,
                                  Dart_NewString(""),
                                  Dart_NewString("getSmi"),
                                  0,
                                  NULL);
   EXPECT_VALID(smi_result);
   Dart_Handle string_result;
   string_result = Dart_InvokeStatic(lib,
                                     Dart_NewString(""),
                                     Dart_NewString("getString"),
                                     0,
                                     NULL);
   EXPECT_VALID(string_result);
   EXPECT(Dart_IsString(string_result));
-  Dart_Handle list_result;
-  list_result = Dart_InvokeStatic(lib,
-                                  Dart_NewString(""),
-                                  Dart_NewString("getList"),
-                                  0,
-                                  NULL);
-  EXPECT_VALID(list_result);
-  EXPECT(Dart_IsList(list_result));
-  intptr_t result_len = 0;
-  EXPECT_VALID(Dart_ListLength(list_result, &result_len));
-  EXPECT_EQ(kArrayLength, result_len);
+
   {
     DARTSCOPE_NOCHECKS(isolate);
 
     {
       uint8_t* buffer;
       SnapshotWriter writer(Snapshot::kMessage, &buffer, &allocator);
       Smi& smi = Smi::Handle();
       smi ^= Api::UnwrapHandle(smi_result);
       writer.WriteObject(smi.raw());
       writer.FinalizeBuffer();
 
       // Read object back from the snapshot into a C structure.
       CMessageReader mreader(buffer + Snapshot::kHeaderSize,
                              writer.BytesWritten(),
                              &allocator);
       Dart_CObject* value = mreader.ReadObject();
+      EXPECT_NOTNULL(value);
       EXPECT_EQ(Dart_CObject::kInt32, value->type);
       EXPECT_EQ(42, value->value.as_int32);
       free(value);
       free(buffer);
     }
     {
       uint8_t* buffer;
       SnapshotWriter writer(Snapshot::kMessage, &buffer, &allocator);
       String& str = String::Handle();
       str ^= Api::UnwrapHandle(string_result);
       writer.WriteObject(str.raw());
       writer.FinalizeBuffer();
 
       // Read object back from the snapshot into a C structure.
       CMessageReader mreader(buffer + Snapshot::kHeaderSize,
                              writer.BytesWritten(),
                              &allocator);
       Dart_CObject* value = mreader.ReadObject();
+      EXPECT_NOTNULL(value);
       EXPECT_EQ(Dart_CObject::kString, value->type);
       EXPECT_STREQ("Hello, world!", value->value.as_string);
       free(value);
       free(buffer);
     }
+  }
+  Dart_ExitScope();
+  Dart_ShutdownIsolate();
+}
+
+
+UNIT_TEST_CASE(DartGeneratedListMessages) {
+  const int kArrayLength = 10;
+  static const char* kScriptChars =
+      "final int kArrayLength = 10;\n"
+      "getList() {\n"
+      "  return new List(kArrayLength);\n"
+      "}\n"
+      "getIntList() {\n"
+      "  var list = new List<int>(kArrayLength);\n"
+      "  for (var i = 0; i < kArrayLength; i++) list[i] = i;\n"
+      "  return list;\n"
+      "}\n"
+      "getStringList() {\n"
+      "  var list = new List<String>(kArrayLength);\n"
+      "  for (var i = 0; i < kArrayLength; i++) list[i] = i.toString();\n"
+      "  return list;\n"
+      "}\n"
+      "getMixedList() {\n"
+      "  var list = new List(kArrayLength);\n"
+      "  list[0] = 0;\n"
+      "  list[1] = '1';\n"
+      "  list[2] = 2.2;\n"
+      "  list[3] = true;\n"
+      "  return list;\n"
+      "}\n";
+
+  TestCase::CreateTestIsolate();
+  Isolate* isolate = Isolate::Current();
+  EXPECT(isolate != NULL);
+  Dart_EnterScope();
+
+  Dart_Handle lib = TestCase::LoadTestScript(kScriptChars, NULL);
+  EXPECT_VALID(lib);
+
+  {
+    DARTSCOPE_NOCHECKS(isolate);
+
     {
-      uint8_t* buffer;
-      SnapshotWriter writer(Snapshot::kMessage, &buffer, &allocator);
-      const Object& list = Object::Handle(Api::UnwrapHandle(list_result));
-      writer.WriteObject(list.raw());
-      writer.FinalizeBuffer();
-      // TODO(sgjesse): Make this work!
-      free(buffer);
+      // Generate a list of nulls from Dart code.
+      Dart_CObject* value = GetDeserializedDartObject(lib, "getList");
+      EXPECT_NOTNULL(value);
+      EXPECT_EQ(Dart_CObject::kArray, value->type);
+      EXPECT_EQ(kArrayLength, value->value.as_array.length);
+      for (int i = 0; i < kArrayLength; i++) {
+        EXPECT_EQ(Dart_CObject::kNull, value->value.as_array.values[i]->type);
+        free(value->value.as_array.values[i]);
+      }
+      free(value);
+    }
+    {
+      // Generate a list of ints from Dart code.
+      Dart_CObject* value = GetDeserializedDartObject(lib, "getIntList");
+      EXPECT_NOTNULL(value);
+      EXPECT_EQ(Dart_CObject::kArray, value->type);
+      EXPECT_EQ(kArrayLength, value->value.as_array.length);
+      for (int i = 0; i < kArrayLength; i++) {
+        EXPECT_EQ(Dart_CObject::kInt32, value->value.as_array.values[i]->type);
+        EXPECT_EQ(i, value->value.as_array.values[i]->value.as_int32);
+        free(value->value.as_array.values[i]);
+      }
+      free(value);
+    }
+    {
+      // Generate a list of strings from Dart code.
+      Dart_CObject* value = GetDeserializedDartObject(lib, "getStringList");
+      EXPECT_NOTNULL(value);
+      EXPECT_EQ(Dart_CObject::kArray, value->type);
+      EXPECT_EQ(kArrayLength, value->value.as_array.length);
+      for (int i = 0; i < kArrayLength; i++) {
+        EXPECT_EQ(Dart_CObject::kString, value->value.as_array.values[i]->type);
+        char buffer[3];
+        snprintf(buffer, sizeof(buffer), "%d", i);
+        EXPECT_STREQ(buffer, value->value.as_array.values[i]->value.as_string);
+        free(value->value.as_array.values[i]);
+      }
+      free(value);
+    }
+    {
+      // Generate a list of objects of different types from Dart code.
+      Dart_CObject* value = GetDeserializedDartObject(lib, "getMixedList");
+      EXPECT_NOTNULL(value);
+      EXPECT_EQ(Dart_CObject::kArray, value->type);
+      EXPECT_EQ(kArrayLength, value->value.as_array.length);
+
+      EXPECT_EQ(Dart_CObject::kInt32, value->value.as_array.values[0]->type);
+      EXPECT_EQ(0, value->value.as_array.values[0]->value.as_int32);
+      EXPECT_EQ(Dart_CObject::kString, value->value.as_array.values[1]->type);
+      EXPECT_STREQ("1", value->value.as_array.values[1]->value.as_string);
+      EXPECT_EQ(Dart_CObject::kDouble, value->value.as_array.values[2]->type);
+      EXPECT_EQ(2.2, value->value.as_array.values[2]->value.as_double);
+      EXPECT_EQ(Dart_CObject::kBool, value->value.as_array.values[3]->type);
+      EXPECT_EQ(true, value->value.as_array.values[3]->value.as_bool);
+
+      for (int i = 0; i < kArrayLength; i++) {
+        if (i > 3) {
+          EXPECT_EQ(Dart_CObject::kNull, value->value.as_array.values[i]->type);
+        }
+        free(value->value.as_array.values[i]);
+      }
+      free(value);
     }
   }
   Dart_ExitScope();
   Dart_ShutdownIsolate();
 }
 
 #endif  // defined(TARGET_ARCH_IA32) || defined(TARGET_ARCH_X64).
 
 }  // namespace dart