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

Side by Side Diff: src/objects-debug.cc

Issue 2144006: Cardmarking writebarrier. (Closed)
Patch Set: change NewSpace and SemiSpace Contains to match HasHeapObjectTag Created 10 years, 7 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
« no previous file with comments | « src/objects.cc ('k') | src/objects-inl.h » ('j') | src/spaces.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2006-2008 the V8 project authors. All rights reserved. 1 // Copyright 2006-2008 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 788 matching lines...) Expand 10 before | Expand all | Expand 10 after
799 PrintF("\n"); 799 PrintF("\n");
800 } 800 }
801 801
802 802
803 void JSGlobalProxy::JSGlobalProxyVerify() { 803 void JSGlobalProxy::JSGlobalProxyVerify() {
804 CHECK(IsJSGlobalProxy()); 804 CHECK(IsJSGlobalProxy());
805 JSObjectVerify(); 805 JSObjectVerify();
806 VerifyObjectField(JSGlobalProxy::kContextOffset); 806 VerifyObjectField(JSGlobalProxy::kContextOffset);
807 // Make sure that this object has no properties, elements. 807 // Make sure that this object has no properties, elements.
808 CHECK_EQ(0, properties()->length()); 808 CHECK_EQ(0, properties()->length());
809 CHECK_EQ(0, elements()->length()); 809 CHECK(HasFastElements());
810 CHECK_EQ(0, FixedArray::cast(elements())->length());
810 } 811 }
811 812
812 813
813 void JSGlobalObject::JSGlobalObjectPrint() { 814 void JSGlobalObject::JSGlobalObjectPrint() {
814 PrintF("global "); 815 PrintF("global ");
815 JSObjectPrint(); 816 JSObjectPrint();
816 PrintF("global context : "); 817 PrintF("global context : ");
817 global_context()->ShortPrint(); 818 global_context()->ShortPrint();
818 PrintF("\n"); 819 PrintF("\n");
819 } 820 }
(...skipping 530 matching lines...) Expand 10 before | Expand all | Expand 10 after
1350 ASSERT(get(i)->IsTheHole()); 1351 ASSERT(get(i)->IsTheHole());
1351 get(i)->Verify(); 1352 get(i)->Verify();
1352 } 1353 }
1353 } 1354 }
1354 } 1355 }
1355 1356
1356 1357
1357 #endif // DEBUG 1358 #endif // DEBUG
1358 1359
1359 } } // namespace v8::internal 1360 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/objects.cc ('k') | src/objects-inl.h » ('j') | src/spaces.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698