Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(375)

Unified Diff: runtime/vm/dart_api_message.cc

Issue 1260033004: Fix for issue 23244 (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: indent-code Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | runtime/vm/raw_object_snapshot.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: runtime/vm/dart_api_message.cc
diff --git a/runtime/vm/dart_api_message.cc b/runtime/vm/dart_api_message.cc
index 2ee1c37bbe5c32cb56f0d8703ce5ccb8dce8a7b6..90604483ae3e71e76aeb18230409ea1295515f07 100644
--- a/runtime/vm/dart_api_message.cc
+++ b/runtime/vm/dart_api_message.cc
@@ -481,6 +481,9 @@ Dart_CObject* ApiMessageReader::ReadObjectRef() {
object_id = NextAvailableObjectId();
}
+ intptr_t tags = ReadTags();
+ USE(tags);
+
// Reading of regular dart instances has limited support in order to
// read typed data views.
if (SerializedHeaderData::decode(class_header) == kInstanceObjectId) {
@@ -502,10 +505,6 @@ Dart_CObject* ApiMessageReader::ReadObjectRef() {
AddBackRef(object_id, value, kIsNotDeserialized);
return value;
}
-
- intptr_t tags = ReadTags();
- USE(tags);
-
return ReadInternalVMObject(class_id, object_id);
}
@@ -1060,6 +1059,7 @@ bool ApiMessageWriter::WriteCObjectRef(Dart_CObject* object) {
WriteInlinedHeader(object);
// Write out the class information.
WriteIndexedObject(kArrayCid);
+ WriteTags(0);
// Write out the length information.
WriteSmi(array_length);
// Add object to forward list so that this object is serialized later.
« no previous file with comments | « no previous file | runtime/vm/raw_object_snapshot.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698