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

Side by Side Diff: src/heap-snapshot-generator.cc

Issue 329463005: Revert 21720: "Introduce FieldIndex to unify and abstract property/field offset" (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 6 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/field-index-inl.h ('k') | src/ia32/stub-cache-ia32.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/v8.h" 5 #include "src/v8.h"
6 6
7 #include "src/heap-snapshot-generator-inl.h" 7 #include "src/heap-snapshot-generator-inl.h"
8 8
9 #include "src/allocation-tracker.h" 9 #include "src/allocation-tracker.h"
10 #include "src/code-stubs.h" 10 #include "src/code-stubs.h"
(...skipping 1642 matching lines...) Expand 10 before | Expand all | Expand 10 after
1653 NULL, 1653 NULL,
1654 js_obj->GetInObjectPropertyOffset(index)); 1654 js_obj->GetInObjectPropertyOffset(index));
1655 } else { 1655 } else {
1656 TagObject(value, "(hidden properties)"); 1656 TagObject(value, "(hidden properties)");
1657 SetInternalReference( 1657 SetInternalReference(
1658 js_obj, entry, 1658 js_obj, entry,
1659 "hidden_properties", value, 1659 "hidden_properties", value,
1660 js_obj->GetInObjectPropertyOffset(index)); 1660 js_obj->GetInObjectPropertyOffset(index));
1661 } 1661 }
1662 } else { 1662 } else {
1663 FieldIndex field_index = 1663 Object* value = js_obj->RawFastPropertyAt(index);
1664 FieldIndex::ForDescriptor(js_obj->map(), i);
1665 Object* value = js_obj->RawFastPropertyAt(field_index);
1666 if (k != heap_->hidden_string()) { 1664 if (k != heap_->hidden_string()) {
1667 SetPropertyReference(js_obj, entry, k, value); 1665 SetPropertyReference(js_obj, entry, k, value);
1668 } else { 1666 } else {
1669 TagObject(value, "(hidden properties)"); 1667 TagObject(value, "(hidden properties)");
1670 SetInternalReference(js_obj, entry, "hidden_properties", value); 1668 SetInternalReference(js_obj, entry, "hidden_properties", value);
1671 } 1669 }
1672 } 1670 }
1673 break; 1671 break;
1674 } 1672 }
1675 case CONSTANT: 1673 case CONSTANT:
(...skipping 1495 matching lines...) Expand 10 before | Expand all | Expand 10 after
3171 writer_->AddString("\"<dummy>\""); 3169 writer_->AddString("\"<dummy>\"");
3172 for (int i = 1; i < sorted_strings.length(); ++i) { 3170 for (int i = 1; i < sorted_strings.length(); ++i) {
3173 writer_->AddCharacter(','); 3171 writer_->AddCharacter(',');
3174 SerializeString(sorted_strings[i]); 3172 SerializeString(sorted_strings[i]);
3175 if (writer_->aborted()) return; 3173 if (writer_->aborted()) return;
3176 } 3174 }
3177 } 3175 }
3178 3176
3179 3177
3180 } } // namespace v8::internal 3178 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/field-index-inl.h ('k') | src/ia32/stub-cache-ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698