Handle TypedDataView objects in Dart_TypedDataAcquireData and

runtime/vm/snapshot.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 "vm/snapshot.h"
 
 #include "platform/assert.h"
 #include "vm/bigint_operations.h"
 #include "vm/bootstrap.h"
 #include "vm/class_finalizer.h"
@@ skipping 36 matching lines @@
 
 static intptr_t ClassIdFromObjectId(intptr_t object_id) {
   ASSERT(object_id > kClassIdsOffset);
   intptr_t class_id = (object_id - kClassIdsOffset);
   return class_id;
 }
 
 
 static intptr_t ObjectIdFromClassId(intptr_t class_id) {
   ASSERT((class_id > kIllegalCid) && (class_id < kNumPredefinedCids));
+  ASSERT(!RawObject::IsTypedDataViewClassId(class_id));
   return (class_id + kClassIdsOffset);
 }
 
 
 static RawType* GetType(ObjectStore* object_store, int index) {
   switch (index) {
     case kObjectType: return object_store->object_type();
     case kNullType: return object_store->null_type();
     case kDynamicType: return object_store->dynamic_type();
     case kVoidType: return object_store->void_type();
@@ skipping 241 matching lines @@
   intptr_t tags = ReadIntptrValue();
   switch (class_id) {
 #define SNAPSHOT_READ(clazz)                                                   \
     case clazz::kClassId: {                                                    \
       obj_ = clazz::ReadFrom(this, object_id, tags, kind_);                    \
       break;                                                                   \
     }
     CLASS_LIST_NO_OBJECT(SNAPSHOT_READ)
 #undef SNAPSHOT_READ
 #define SNAPSHOT_READ(clazz)                                                   \
-    case kTypedData##clazz##Cid: {                                             \
-      obj_ = TypedData::ReadFrom(this, object_id, tags, kind_);                \
-      break;                                                                   \
-    }                                                                          \
-    case kExternalTypedData##clazz##Cid: {                                     \
-      obj_ = ExternalTypedData::ReadFrom(this, object_id, tags, kind_);        \
-      break;                                                                   \
+    case kTypedData##clazz##Cid:                                               \
+
+    CLASS_LIST_TYPED_DATA(SNAPSHOT_READ) {
+      obj_ = TypedData::ReadFrom(this, object_id, tags, kind_);
+      break;
     }
-    CLASS_LIST_TYPED_DATA(SNAPSHOT_READ)
+#undef SNAPSHOT_READ
+#define SNAPSHOT_READ(clazz)                                                   \
+    case kExternalTypedData##clazz##Cid:                                       \
+
+    CLASS_LIST_TYPED_DATA(SNAPSHOT_READ) {
+      obj_ = ExternalTypedData::ReadFrom(this, object_id, tags, kind_);
+      break;
+    }
 #undef SNAPSHOT_READ
     default: UNREACHABLE(); break;
   }
   if (kind_ == Snapshot::kFull) {
     obj_.SetCreatedFromSnapshot();
   }
   return obj_.raw();
 }
 
 
@@ skipping 459 matching lines @@
   ASSERT(!cls_.IsNull());
   switch (cls_.id()) {
 #define SNAPSHOT_READ(clazz)                                                   \
     case clazz::kClassId: {                                                    \
       obj_ = clazz::ReadFrom(this, object_id, tags, kind_);                    \
       break;                                                                   \
     }
     CLASS_LIST_NO_OBJECT(SNAPSHOT_READ)
 #undef SNAPSHOT_READ
 #define SNAPSHOT_READ(clazz)                                                   \
-    case kTypedData##clazz##Cid: {                                             \
-      obj_ = TypedData::ReadFrom(this, object_id, tags, kind_);                \
-      break;                                                                   \
-    }                                                                          \
-    case kExternalTypedData##clazz##Cid: {                                     \
-      obj_ = ExternalTypedData::ReadFrom(this, object_id, tags, kind_);        \
-      break;                                                                   \
+    case kTypedData##clazz##Cid:                                               \
+
+    CLASS_LIST_TYPED_DATA(SNAPSHOT_READ) {
+      obj_ = TypedData::ReadFrom(this, object_id, tags, kind_);
+      break;
     }
-    CLASS_LIST_TYPED_DATA(SNAPSHOT_READ)
+#undef SNAPSHOT_READ
+#define SNAPSHOT_READ(clazz)                                                   \
+    case kExternalTypedData##clazz##Cid:                                       \
+
+    CLASS_LIST_TYPED_DATA(SNAPSHOT_READ) {
+      obj_ = ExternalTypedData::ReadFrom(this, object_id, tags, kind_);
+      break;
+    }
 #undef SNAPSHOT_READ
     default: UNREACHABLE(); break;
   }
   if (kind_ == Snapshot::kFull) {
     obj_.SetCreatedFromSnapshot();
   }
   return obj_.raw();
 }
 
 
@@ skipping 94 matching lines @@
   // First check if object can be written as a simple predefined type.
   if (CheckAndWritePredefinedObject(raw)) {
     return;
   }
 
   NoGCScope no_gc;
   RawClass* cls = class_table_->At(raw->GetClassId());
   intptr_t class_id = cls->ptr()->id_;
   ASSERT(class_id == raw->GetClassId());
   if (class_id >= kNumPredefinedCids) {
-    if (Class::IsSignatureClass(cls)) {
-      // We do not allow closure objects in an isolate message.
-      set_exception_type(Exceptions::kArgument);
-      // TODO(6726): Allocate these constant strings once in the VM isolate.
-      set_exception_msg("Illegal argument in isolate message"
-                        " : (object is a closure)");
-      Isolate::Current()->long_jump_base()->Jump(1, *ErrorHandle());
-    }
-    // Object is being referenced, add it to the forward ref list and mark
-    // it so that future references to this object in the snapshot will use
-    // this object id. Mark it as not having been serialized yet so that we
-    // will serialize the object when we go through the forward list.
-    intptr_t object_id = MarkObject(raw, kIsNotSerialized);
-
-    // Write out the serialization header value for this object.
-    WriteInlinedObjectHeader(object_id);
-
-    // Indicate this is an instance object.
-    WriteIntptrValue(SerializedHeaderData::encode(kInstanceObjectId));
-
-    // Write out the class information for this object.
-    WriteObjectImpl(cls);
-
+    WriteInstanceRef(raw, cls);
     return;
   }
   if (class_id == kArrayCid) {
     // Object is being referenced, add it to the forward ref list and mark
     // it so that future references to this object in the snapshot will use
     // this object id. Mark it as not having been serialized yet so that we
     // will serialize the object when we go through the forward list.
     intptr_t object_id = MarkObject(raw, kIsNotSerialized);
 
     RawArray* rawarray = reinterpret_cast<RawArray*>(raw);
@@ skipping 38 matching lines @@
 #define SNAPSHOT_WRITE(clazz)                                                  \
     case clazz::kClassId: {                                                    \
       Raw##clazz* raw_obj = reinterpret_cast<Raw##clazz*>(raw);                \
       raw_obj->WriteTo(this, object_id, kind_);                                \
       return;                                                                  \
     }                                                                          \
 
     CLASS_LIST_NO_OBJECT(SNAPSHOT_WRITE)
 #undef SNAPSHOT_WRITE
 #define SNAPSHOT_WRITE(clazz)                                                  \
-    case kTypedData##clazz##Cid: {                                             \
-      RawTypedData* raw_obj = reinterpret_cast<RawTypedData*>(raw);            \
-      raw_obj->WriteTo(this, object_id, kind_);                                \
-      return;                                                                  \
-    }                                                                          \
-    case kExternalTypedData##clazz##Cid: {                                     \
-      RawExternalTypedData* raw_obj =                                          \
-        reinterpret_cast<RawExternalTypedData*>(raw);                          \
-      raw_obj->WriteTo(this, object_id, kind_);                                \
-      return;                                                                  \
-    }                                                                          \
+    case kTypedData##clazz##Cid:                                               \
+
+    CLASS_LIST_TYPED_DATA(SNAPSHOT_WRITE) {
+      RawTypedData* raw_obj = reinterpret_cast<RawTypedData*>(raw);
+      raw_obj->WriteTo(this, object_id, kind_);
+      return;
+    }
+#undef SNAPSHOT_WRITE
+#define SNAPSHOT_WRITE(clazz)                                                  \
+    case kExternalTypedData##clazz##Cid:                                       \
+
+    CLASS_LIST_TYPED_DATA(SNAPSHOT_WRITE) {
+      RawExternalTypedData* raw_obj =
+        reinterpret_cast<RawExternalTypedData*>(raw);
+      raw_obj->WriteTo(this, object_id, kind_);
+      return;
+    }
+#undef SNAPSHOT_WRITE
+#define SNAPSHOT_WRITE(clazz)                                                  \
+    case kTypedData##clazz##ViewCid:                                           \
 
     CLASS_LIST_TYPED_DATA(SNAPSHOT_WRITE)
+    case kByteDataViewCid: {
+      WriteInstanceRef(raw, cls);
+      return;
+    }
 #undef SNAPSHOT_WRITE
     default: break;
   }
   UNREACHABLE();
 }
 
 
 void FullSnapshotWriter::WriteFullSnapshot() {
   Isolate* isolate = Isolate::Current();
   ASSERT(isolate != NULL);
@@ skipping 158 matching lines @@
   //    ......
   NoGCScope no_gc;
   uword tags = raw->ptr()->tags_;
   ASSERT(SerializedHeaderTag::decode(tags) == kObjectId);
   intptr_t object_id = SerializedHeaderData::decode(tags);
   tags = forward_list_[object_id - kMaxPredefinedObjectIds]->tags();
   RawClass* cls = class_table_->At(RawObject::ClassIdTag::decode(tags));
   intptr_t class_id = cls->ptr()->id_;
 
   if (class_id >= kNumPredefinedCids) {
-    if (Class::IsSignatureClass(cls)) {
-      // We do not allow closure objects in an isolate message.
-      set_exception_type(Exceptions::kArgument);
-      // TODO(6726): Allocate these constant strings once in the VM isolate.
-      set_exception_msg("Illegal argument in isolate message"
-                        " : (object is a closure)");
-      Isolate::Current()->long_jump_base()->Jump(1, *ErrorHandle());
-    }
-    if (cls->ptr()->num_native_fields_ != 0) {
-      // We do not allow objects with native fields in an isolate message.
-      set_exception_type(Exceptions::kArgument);
-      // TODO(6726): Allocate these constant strings once in the VM isolate.
-      set_exception_msg("Illegal argument in isolate message"
-                        " : (object extends NativeWrapper)");
-
-      Isolate::Current()->long_jump_base()->Jump(1, *ErrorHandle());
-    }
-    // Object is regular dart instance.
-    intptr_t instance_size =
-        cls->ptr()->instance_size_in_words_ << kWordSizeLog2;
-    ASSERT(instance_size != 0);
-
-    // Write out the serialization header value for this object.
-    WriteInlinedObjectHeader(object_id);
-
-    // Indicate this is an instance object.
-    WriteIntptrValue(SerializedHeaderData::encode(kInstanceObjectId));
-
-    // Write out the tags.
-    WriteIntptrValue(tags);
-
-    // Write out the class information for this object.
-    WriteObjectImpl(cls);
-
-    // Write out all the fields for the object.
-    intptr_t offset = Object::InstanceSize();
-    while (offset < instance_size) {
-      WriteObjectRef(*reinterpret_cast<RawObject**>(
-          reinterpret_cast<uword>(raw->ptr()) + offset));
-      offset += kWordSize;
-    }
+    WriteInstance(object_id, raw, cls, tags);
     return;
   }
   switch (class_id) {
 #define SNAPSHOT_WRITE(clazz)                                                  \
     case clazz::kClassId: {                                                    \
       Raw##clazz* raw_obj = reinterpret_cast<Raw##clazz*>(raw);                \
       raw_obj->WriteTo(this, object_id, kind_);                                \
       return;                                                                  \
     }                                                                          \
 
     CLASS_LIST_NO_OBJECT(SNAPSHOT_WRITE)
 #undef SNAPSHOT_WRITE
 #define SNAPSHOT_WRITE(clazz)                                                  \
-    case kTypedData##clazz##Cid: {                                             \
-      RawTypedData* raw_obj = reinterpret_cast<RawTypedData*>(raw);            \
-      raw_obj->WriteTo(this, object_id, kind_);                                \
-      return;                                                                  \
-    }                                                                          \
-    case kExternalTypedData##clazz##Cid: {                                     \
-      RawExternalTypedData* raw_obj =                                          \
-        reinterpret_cast<RawExternalTypedData*>(raw);                          \
-      raw_obj->WriteTo(this, object_id, kind_);                                \
-      return;                                                                  \
-    }                                                                          \
+    case kTypedData##clazz##Cid:                                               \
+
+    CLASS_LIST_TYPED_DATA(SNAPSHOT_WRITE) {
+      RawTypedData* raw_obj = reinterpret_cast<RawTypedData*>(raw);
+      raw_obj->WriteTo(this, object_id, kind_);
+      return;
+    }
+#undef SNAPSHOT_WRITE
+#define SNAPSHOT_WRITE(clazz)                                                  \
+    case kExternalTypedData##clazz##Cid:                                       \
+
+    CLASS_LIST_TYPED_DATA(SNAPSHOT_WRITE) {
+      RawExternalTypedData* raw_obj =
+        reinterpret_cast<RawExternalTypedData*>(raw);
+      raw_obj->WriteTo(this, object_id, kind_);
+      return;
+    }
+#undef SNAPSHOT_WRITE
+#define SNAPSHOT_WRITE(clazz)                                                  \
+    case kTypedData##clazz##ViewCid:                                           \
 
     CLASS_LIST_TYPED_DATA(SNAPSHOT_WRITE)
+    case kByteDataViewCid: {
+      WriteInstance(object_id, raw, cls, tags);
+      return;
+    }
 #undef SNAPSHOT_WRITE
     default: break;
   }
   UNREACHABLE();
 }
 
 
 void SnapshotWriter::WriteForwardedObjects() {
   // Write out all objects that were added to the forward list and have
   // not been serialized yet. These would typically be fields of instance
@@ skipping 59 matching lines @@
   // Write out the type arguments.
   WriteObjectImpl(type_arguments);
 
   // Write out the individual object ids.
   for (intptr_t i = 0; i < len; i++) {
     WriteObjectRef(data[i]);
   }
 }
 
 
+void SnapshotWriter::CheckIfSerializable(RawClass* cls) {
+  if (Class::IsSignatureClass(cls)) {
+    // We do not allow closure objects in an isolate message.
+    set_exception_type(Exceptions::kArgument);
+    // TODO(6726): Allocate these constant strings once in the VM isolate.
+    set_exception_msg("Illegal argument in isolate message"
+                      " : (object is a closure)");
+    Isolate::Current()->long_jump_base()->Jump(1, *ErrorHandle());
+  }
+  if (cls->ptr()->num_native_fields_ != 0) {
+    // We do not allow objects with native fields in an isolate message.
+    set_exception_type(Exceptions::kArgument);
+    // TODO(6726): Allocate these constant strings once in the VM isolate.
+    set_exception_msg("Illegal argument in isolate message"
+                      " : (object extends NativeWrapper)");
+
+    Isolate::Current()->long_jump_base()->Jump(1, *ErrorHandle());
+  }
+}
+
+
+void SnapshotWriter::WriteInstance(intptr_t object_id,
+                                   RawObject* raw,
+                                   RawClass* cls,
+                                   intptr_t tags) {
+  // First check if object is a closure or has native fields.
+  CheckIfSerializable(cls);
+
+  // Object is regular dart instance.
+  intptr_t instance_size =
+      cls->ptr()->instance_size_in_words_ << kWordSizeLog2;
+  ASSERT(instance_size != 0);
+
+  // Write out the serialization header value for this object.
+  WriteInlinedObjectHeader(object_id);
+
+  // Indicate this is an instance object.
+  WriteIntptrValue(SerializedHeaderData::encode(kInstanceObjectId));
+
+  // Write out the tags.
+  WriteIntptrValue(tags);
+
+  // Write out the class information for this object.
+  WriteObjectImpl(cls);
+
+  // Write out all the fields for the object.
+  intptr_t offset = Object::InstanceSize();
+  while (offset < instance_size) {
+    WriteObjectRef(*reinterpret_cast<RawObject**>(
+        reinterpret_cast<uword>(raw->ptr()) + offset));
+    offset += kWordSize;
+  }
+  return;
+}
+
+
+void SnapshotWriter::WriteInstanceRef(RawObject* raw, RawClass* cls) {
+  // First check if object is a closure or has native fields.
+  CheckIfSerializable(cls);
+
+  // Object is being referenced, add it to the forward ref list and mark
+  // it so that future references to this object in the snapshot will use
+  // this object id. Mark it as not having been serialized yet so that we
+  // will serialize the object when we go through the forward list.
+  intptr_t object_id = MarkObject(raw, kIsNotSerialized);
+
+  // Write out the serialization header value for this object.
+  WriteInlinedObjectHeader(object_id);
+
+  // Indicate this is an instance object.
+  WriteIntptrValue(SerializedHeaderData::encode(kInstanceObjectId));
+
+  // Write out the class information for this object.
+  WriteObjectImpl(cls);
+}
+
+
 void SnapshotWriter::ThrowException(Exceptions::ExceptionType type,
                                     const char* msg) {
   Isolate::Current()->object_store()->clear_sticky_error();
   UnmarkAll();
   const String& msg_obj = String::Handle(String::New(msg));
   const Array& args = Array::Handle(Array::New(1));
   args.SetAt(0, msg_obj);
   Exceptions::ThrowByType(type, args);
   UNREACHABLE();
 }
@@ skipping 57 matching lines @@
     UnmarkAll();
     isolate->set_long_jump_base(base);
   } else {
     isolate->set_long_jump_base(base);
     ThrowException(exception_type(), exception_msg());
   }
 }
 
 
 }  // namespace dart