| OLD | NEW |
| 1 // Copyright 2011 the V8 project authors. All rights reserved. | 1 // Copyright 2014 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 |
| 11 // with the distribution. | 11 // with the distribution. |
| 12 // * Neither the name of Google Inc. nor the names of its | 12 // * Neither the name of Google Inc. nor the names of its |
| 13 // contributors may be used to endorse or promote products derived | 13 // contributors may be used to endorse or promote products derived |
| 14 // from this software without specific prior written permission. | 14 // from this software without specific prior written permission. |
| 15 // | 15 // |
| 16 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS | 16 // THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS |
| 17 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT | 17 // "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT |
| 18 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR | 18 // LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR |
| 19 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT | 19 // A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT |
| 20 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, | 20 // OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, |
| 21 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT | 21 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT |
| 22 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, | 22 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, |
| 23 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY | 23 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY |
| 24 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT | 24 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT |
| 25 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE | 25 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE |
| 26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. | 26 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. |
| 27 | 27 |
| 28 #include <stdlib.h> |
| 29 |
| 28 #include "v8.h" | 30 #include "v8.h" |
| 29 | 31 |
| 30 #include "api.h" | 32 #include "cctest.h" |
| 31 #include "debug.h" | |
| 32 #include "execution.h" | |
| 33 #include "factory.h" | 33 #include "factory.h" |
| 34 #include "macro-assembler.h" | 34 |
| 35 #include "objects.h" | 35 #include "close-table.h" |
| 36 #include "global-handles.h" | 36 |
| 37 #include "cctest.h" | 37 namespace { |
| 38 | 38 |
| 39 using namespace v8::internal; | 39 using namespace v8::internal; |
| 40 | 40 |
| 41 TEST(Set) { |
| 42 LocalContext context; |
| 43 Isolate* isolate = CcTest::i_isolate(); |
| 44 Factory* factory = isolate->factory(); |
| 45 HandleScope scope(isolate); |
| 46 Handle<CloseTableSet> close_set = factory->NewCloseTableSet(); |
| 47 CHECK_EQ(2, close_set->NumberOfBuckets()); |
| 48 CHECK_EQ(0, close_set->NumberOfElements()); |
| 49 CHECK_EQ(0, close_set->NumberOfDeletedElements()); |
| 50 |
| 51 Handle<Map> map = factory->NewMap(JS_OBJECT_TYPE, JSObject::kHeaderSize); |
| 52 Handle<JSObject> obj = factory->NewJSObjectFromMap(map); |
| 53 CHECK(!close_set->Contains(*obj)); |
| 54 close_set = CloseTableSet::Add(close_set, obj); |
| 55 CHECK_EQ(1, close_set->NumberOfElements()); |
| 56 CHECK(close_set->Contains(*obj)); |
| 57 close_set = CloseTableSet::Remove(close_set, obj); |
| 58 CHECK_EQ(0, close_set->NumberOfElements()); |
| 59 CHECK(!close_set->Contains(*obj)); |
| 60 |
| 61 // Test for collisions/chaining |
| 62 Handle<JSObject> obj1 = factory->NewJSObjectFromMap(map); |
| 63 close_set = CloseTableSet::Add(close_set, obj1); |
| 64 Handle<JSObject> obj2 = factory->NewJSObjectFromMap(map); |
| 65 close_set = CloseTableSet::Add(close_set, obj2); |
| 66 Handle<JSObject> obj3 = factory->NewJSObjectFromMap(map); |
| 67 close_set = CloseTableSet::Add(close_set, obj3); |
| 68 CHECK_EQ(3, close_set->NumberOfElements()); |
| 69 CHECK(close_set->Contains(*obj1)); |
| 70 CHECK(close_set->Contains(*obj2)); |
| 71 CHECK(close_set->Contains(*obj3)); |
| 72 |
| 73 // Test growth |
| 74 close_set = CloseTableSet::Add(close_set, obj); |
| 75 Handle<JSObject> obj4 = factory->NewJSObjectFromMap(map); |
| 76 close_set = CloseTableSet::Add(close_set, obj4); |
| 77 CHECK(close_set->Contains(*obj)); |
| 78 CHECK(close_set->Contains(*obj1)); |
| 79 CHECK(close_set->Contains(*obj2)); |
| 80 CHECK(close_set->Contains(*obj3)); |
| 81 CHECK(close_set->Contains(*obj4)); |
| 82 CHECK_EQ(5, close_set->NumberOfElements()); |
| 83 CHECK_EQ(0, close_set->NumberOfDeletedElements()); |
| 84 CHECK_EQ(4, close_set->NumberOfBuckets()); |
| 85 |
| 86 // Test shrinking |
| 87 close_set = CloseTableSet::Remove(close_set, obj); |
| 88 close_set = CloseTableSet::Remove(close_set, obj1); |
| 89 close_set = CloseTableSet::Remove(close_set, obj2); |
| 90 close_set = CloseTableSet::Remove(close_set, obj3); |
| 91 CHECK_EQ(1, close_set->NumberOfElements()); |
| 92 CHECK_EQ(2, close_set->NumberOfBuckets()); |
| 93 } |
| 94 |
| 95 TEST(Map) { |
| 96 LocalContext context; |
| 97 Isolate* isolate = CcTest::i_isolate(); |
| 98 Factory* factory = isolate->factory(); |
| 99 HandleScope scope(isolate); |
| 100 Handle<CloseTableMap> close_map = factory->NewCloseTableMap(); |
| 101 CHECK_EQ(2, close_map->NumberOfBuckets()); |
| 102 CHECK_EQ(0, close_map->NumberOfElements()); |
| 103 CHECK_EQ(0, close_map->NumberOfDeletedElements()); |
| 104 |
| 105 Handle<Map> map = factory->NewMap(JS_OBJECT_TYPE, JSObject::kHeaderSize); |
| 106 Handle<JSObject> obj = factory->NewJSObjectFromMap(map); |
| 107 Handle<JSObject> val = factory->NewJSObjectFromMap(map); |
| 108 CHECK(close_map->Lookup(*obj)->IsTheHole()); |
| 109 close_map = CloseTableMap::Put(close_map, obj, val); |
| 110 CHECK_EQ(1, close_map->NumberOfElements()); |
| 111 CHECK(close_map->Lookup(*obj)->SameValue(*val)); |
| 112 close_map = CloseTableMap::Put(close_map, obj, factory->the_hole_value()); |
| 113 CHECK_EQ(0, close_map->NumberOfElements()); |
| 114 CHECK(close_map->Lookup(*obj)->IsTheHole()); |
| 115 |
| 116 // Test for collisions/chaining |
| 117 Handle<JSObject> obj1 = factory->NewJSObjectFromMap(map); |
| 118 Handle<JSObject> obj2 = factory->NewJSObjectFromMap(map); |
| 119 Handle<JSObject> obj3 = factory->NewJSObjectFromMap(map); |
| 120 Handle<JSObject> val1 = factory->NewJSObjectFromMap(map); |
| 121 Handle<JSObject> val2 = factory->NewJSObjectFromMap(map); |
| 122 Handle<JSObject> val3 = factory->NewJSObjectFromMap(map); |
| 123 close_map = CloseTableMap::Put(close_map, obj1, val1); |
| 124 close_map = CloseTableMap::Put(close_map, obj2, val2); |
| 125 close_map = CloseTableMap::Put(close_map, obj3, val3); |
| 126 CHECK_EQ(3, close_map->NumberOfElements()); |
| 127 CHECK(close_map->Lookup(*obj1)->SameValue(*val1)); |
| 128 CHECK(close_map->Lookup(*obj2)->SameValue(*val2)); |
| 129 CHECK(close_map->Lookup(*obj3)->SameValue(*val3)); |
| 130 |
| 131 // Test growth |
| 132 close_map = CloseTableMap::Put(close_map, obj, val); |
| 133 Handle<JSObject> obj4 = factory->NewJSObjectFromMap(map); |
| 134 Handle<JSObject> val4 = factory->NewJSObjectFromMap(map); |
| 135 close_map = CloseTableMap::Put(close_map, obj4, val4); |
| 136 CHECK(close_map->Lookup(*obj)->SameValue(*val)); |
| 137 CHECK(close_map->Lookup(*obj1)->SameValue(*val1)); |
| 138 CHECK(close_map->Lookup(*obj2)->SameValue(*val2)); |
| 139 CHECK(close_map->Lookup(*obj3)->SameValue(*val3)); |
| 140 CHECK(close_map->Lookup(*obj4)->SameValue(*val4)); |
| 141 CHECK_EQ(5, close_map->NumberOfElements()); |
| 142 CHECK_EQ(4, close_map->NumberOfBuckets()); |
| 143 |
| 144 // Test shrinking |
| 145 close_map = CloseTableMap::Put(close_map, obj, factory->the_hole_value()); |
| 146 close_map = CloseTableMap::Put(close_map, obj1, factory->the_hole_value()); |
| 147 close_map = CloseTableMap::Put(close_map, obj2, factory->the_hole_value()); |
| 148 close_map = CloseTableMap::Put(close_map, obj3, factory->the_hole_value()); |
| 149 CHECK_EQ(1, close_map->NumberOfElements()); |
| 150 CHECK_EQ(2, close_map->NumberOfBuckets()); |
| 151 } |
| 152 |
| 153 // --------------------------------------------------------- |
| 154 // BELOW THIS LINE TESTS WERE CONVERTED FROM test-dictionary |
| 155 // --------------------------------------------------------- |
| 41 | 156 |
| 42 TEST(ObjectHashTable) { | 157 TEST(ObjectHashTable) { |
| 43 LocalContext context; | 158 LocalContext context; |
| 44 Isolate* isolate = CcTest::i_isolate(); | 159 Isolate* isolate = CcTest::i_isolate(); |
| 45 Factory* factory = isolate->factory(); | 160 Factory* factory = isolate->factory(); |
| 46 v8::HandleScope scope(context->GetIsolate()); | 161 v8::HandleScope scope(context->GetIsolate()); |
| 47 Handle<ObjectHashTable> table = factory->NewObjectHashTable(23); | 162 Handle<CloseTableMap> table = factory->NewCloseTableMap(); |
| 48 Handle<JSObject> a = factory->NewJSArray(7); | 163 Handle<JSObject> a = factory->NewJSArray(7); |
| 49 Handle<JSObject> b = factory->NewJSArray(11); | 164 Handle<JSObject> b = factory->NewJSArray(11); |
| 50 table = ObjectHashTable::Put(table, a, b); | 165 table = CloseTableMap::Put(table, a, b); |
| 51 CHECK_EQ(table->NumberOfElements(), 1); | 166 CHECK_EQ(table->NumberOfElements(), 1); |
| 52 CHECK_EQ(table->Lookup(*a), *b); | 167 CHECK_EQ(table->Lookup(*a), *b); |
| 53 CHECK_EQ(table->Lookup(*b), CcTest::heap()->the_hole_value()); | 168 CHECK_EQ(table->Lookup(*b), CcTest::heap()->the_hole_value()); |
| 54 | 169 |
| 55 // Keys still have to be valid after objects were moved. | 170 // Keys still have to be valid after objects were moved. |
| 56 CcTest::heap()->CollectGarbage(NEW_SPACE); | 171 CcTest::heap()->CollectGarbage(NEW_SPACE); |
| 57 CHECK_EQ(table->NumberOfElements(), 1); | 172 CHECK_EQ(table->NumberOfElements(), 1); |
| 58 CHECK_EQ(table->Lookup(*a), *b); | 173 CHECK_EQ(table->Lookup(*a), *b); |
| 59 CHECK_EQ(table->Lookup(*b), CcTest::heap()->the_hole_value()); | 174 CHECK_EQ(table->Lookup(*b), CcTest::heap()->the_hole_value()); |
| 60 | 175 |
| 61 // Keys that are overwritten should not change number of elements. | 176 // Keys that are overwritten should not change number of elements. |
| 62 table = ObjectHashTable::Put(table, a, factory->NewJSArray(13)); | 177 table = CloseTableMap::Put(table, a, factory->NewJSArray(13)); |
| 63 CHECK_EQ(table->NumberOfElements(), 1); | 178 CHECK_EQ(table->NumberOfElements(), 1); |
| 64 CHECK_NE(table->Lookup(*a), *b); | 179 CHECK_NE(table->Lookup(*a), *b); |
| 65 | 180 |
| 66 // Keys mapped to the hole should be removed permanently. | 181 // Keys mapped to the hole should be removed permanently. |
| 67 table = ObjectHashTable::Put(table, a, factory->the_hole_value()); | 182 table = CloseTableMap::Put(table, a, factory->the_hole_value()); |
| 68 CHECK_EQ(table->NumberOfElements(), 0); | 183 CHECK_EQ(table->NumberOfElements(), 0); |
| 69 CHECK_EQ(table->NumberOfDeletedElements(), 1); | 184 //CHECK_EQ(table->NumberOfDeletedElements(), 1); |
| 70 CHECK_EQ(table->Lookup(*a), CcTest::heap()->the_hole_value()); | 185 CHECK_EQ(table->Lookup(*a), CcTest::heap()->the_hole_value()); |
| 71 | 186 |
| 72 // Keys should map back to their respective values and also should get | 187 // Keys should map back to their respective values and also should get |
| 73 // an identity hash code generated. | 188 // an identity hash code generated. |
| 74 for (int i = 0; i < 100; i++) { | 189 for (int i = 0; i < 100; i++) { |
| 75 Handle<JSReceiver> key = factory->NewJSArray(7); | 190 Handle<JSReceiver> key = factory->NewJSArray(7); |
| 76 Handle<JSObject> value = factory->NewJSArray(11); | 191 Handle<JSObject> value = factory->NewJSArray(11); |
| 77 table = ObjectHashTable::Put(table, key, value); | 192 table = CloseTableMap::Put(table, key, value); |
| 78 CHECK_EQ(table->NumberOfElements(), i + 1); | 193 CHECK_EQ(table->NumberOfElements(), i + 1); |
| 79 CHECK_NE(table->FindEntry(*key), ObjectHashTable::kNotFound); | 194 CHECK_NE(table->FindEntry(*key), CloseTableMap::kNotFound); |
| 80 CHECK_EQ(table->Lookup(*key), *value); | 195 CHECK_EQ(table->Lookup(*key), *value); |
| 81 CHECK(key->GetIdentityHash()->IsSmi()); | 196 CHECK(key->GetIdentityHash()->IsSmi()); |
| 82 } | 197 } |
| 83 | 198 |
| 84 // Keys never added to the map which already have an identity hash | 199 // Keys never added to the map which already have an identity hash |
| 85 // code should not be found. | 200 // code should not be found. |
| 86 for (int i = 0; i < 100; i++) { | 201 for (int i = 0; i < 100; i++) { |
| 87 Handle<JSReceiver> key = factory->NewJSArray(7); | 202 Handle<JSReceiver> key = factory->NewJSArray(7); |
| 88 CHECK(JSReceiver::GetOrCreateIdentityHash(key)->IsSmi()); | 203 CHECK(JSReceiver::GetOrCreateIdentityHash(key)->IsSmi()); |
| 89 CHECK_EQ(table->FindEntry(*key), ObjectHashTable::kNotFound); | 204 CHECK_EQ(table->FindEntry(*key), CloseTableMap::kNotFound); |
| 90 CHECK_EQ(table->Lookup(*key), CcTest::heap()->the_hole_value()); | 205 CHECK_EQ(table->Lookup(*key), CcTest::heap()->the_hole_value()); |
| 91 CHECK(key->GetIdentityHash()->IsSmi()); | 206 CHECK(key->GetIdentityHash()->IsSmi()); |
| 92 } | 207 } |
| 93 | 208 |
| 94 // Keys that don't have an identity hash should not be found and also | 209 // Keys that don't have an identity hash should not be found and also |
| 95 // should not get an identity hash code generated. | 210 // should not get an identity hash code generated. |
| 96 for (int i = 0; i < 100; i++) { | 211 for (int i = 0; i < 100; i++) { |
| 97 Handle<JSReceiver> key = factory->NewJSArray(7); | 212 Handle<JSReceiver> key = factory->NewJSArray(7); |
| 98 CHECK_EQ(table->Lookup(*key), CcTest::heap()->the_hole_value()); | 213 CHECK_EQ(table->Lookup(*key), CcTest::heap()->the_hole_value()); |
| 99 CHECK_EQ(key->GetIdentityHash(), | 214 CHECK_EQ(key->GetIdentityHash(), |
| 100 CcTest::heap()->undefined_value()); | 215 CcTest::heap()->undefined_value()); |
| 101 } | 216 } |
| 102 } | 217 } |
| 103 | 218 |
| 104 | 219 |
| 105 class ObjectHashTableTest: public ObjectHashTable { | |
| 106 public: | |
| 107 void insert(int entry, int key, int value) { | |
| 108 set(EntryToIndex(entry), Smi::FromInt(key)); | |
| 109 set(EntryToIndex(entry) + 1, Smi::FromInt(value)); | |
| 110 } | |
| 111 | |
| 112 int lookup(int key) { | |
| 113 return Smi::cast(Lookup(Smi::FromInt(key)))->value(); | |
| 114 } | |
| 115 | |
| 116 int capacity() { | |
| 117 return Capacity(); | |
| 118 } | |
| 119 }; | |
| 120 | |
| 121 | |
| 122 TEST(HashTableRehash) { | |
| 123 LocalContext context; | |
| 124 Isolate* isolate = CcTest::i_isolate(); | |
| 125 Factory* factory = isolate->factory(); | |
| 126 v8::HandleScope scope(context->GetIsolate()); | |
| 127 // Test almost filled table. | |
| 128 { | |
| 129 Handle<ObjectHashTable> table = factory->NewObjectHashTable(100); | |
| 130 ObjectHashTableTest* t = reinterpret_cast<ObjectHashTableTest*>(*table); | |
| 131 int capacity = t->capacity(); | |
| 132 for (int i = 0; i < capacity - 1; i++) { | |
| 133 t->insert(i, i * i, i); | |
| 134 } | |
| 135 t->Rehash(Smi::FromInt(0)); | |
| 136 for (int i = 0; i < capacity - 1; i++) { | |
| 137 CHECK_EQ(i, t->lookup(i * i)); | |
| 138 } | |
| 139 } | |
| 140 // Test half-filled table. | |
| 141 { | |
| 142 Handle<ObjectHashTable> table = factory->NewObjectHashTable(100); | |
| 143 ObjectHashTableTest* t = reinterpret_cast<ObjectHashTableTest*>(*table); | |
| 144 int capacity = t->capacity(); | |
| 145 for (int i = 0; i < capacity / 2; i++) { | |
| 146 t->insert(i, i * i, i); | |
| 147 } | |
| 148 t->Rehash(Smi::FromInt(0)); | |
| 149 for (int i = 0; i < capacity / 2; i++) { | |
| 150 CHECK_EQ(i, t->lookup(i * i)); | |
| 151 } | |
| 152 } | |
| 153 } | |
| 154 | |
| 155 | |
| 156 #ifdef DEBUG | 220 #ifdef DEBUG |
| 157 TEST(ObjectHashSetCausesGC) { | 221 TEST(CloseTableSetCausesGC) { |
| 158 i::FLAG_stress_compaction = false; | 222 i::FLAG_stress_compaction = false; |
| 159 LocalContext context; | 223 LocalContext context; |
| 160 Isolate* isolate = CcTest::i_isolate(); | 224 Isolate* isolate = CcTest::i_isolate(); |
| 161 Factory* factory = isolate->factory(); | 225 Factory* factory = isolate->factory(); |
| 162 v8::HandleScope scope(context->GetIsolate()); | 226 v8::HandleScope scope(context->GetIsolate()); |
| 163 Handle<ObjectHashSet> table = factory->NewObjectHashSet(1); | 227 Handle<CloseTableSet> table = factory->NewCloseTableSet(); |
| 164 Handle<JSObject> key = factory->NewJSArray(0); | 228 Handle<JSObject> key = factory->NewJSArray(0); |
| 165 v8::Handle<v8::Object> key_obj = v8::Utils::ToLocal(key); | 229 v8::Handle<v8::Object> key_obj = v8::Utils::ToLocal(key); |
| 166 | 230 |
| 167 // Force allocation of hash table backing store for hidden properties. | 231 // Force allocation of hash table backing store for hidden properties. |
| 168 key_obj->SetHiddenValue(v8_str("key 1"), v8_str("val 1")); | 232 key_obj->SetHiddenValue(v8_str("key 1"), v8_str("val 1")); |
| 169 key_obj->SetHiddenValue(v8_str("key 2"), v8_str("val 2")); | 233 key_obj->SetHiddenValue(v8_str("key 2"), v8_str("val 2")); |
| 170 key_obj->SetHiddenValue(v8_str("key 3"), v8_str("val 3")); | 234 key_obj->SetHiddenValue(v8_str("key 3"), v8_str("val 3")); |
| 171 | 235 |
| 172 // Simulate a full heap so that generating an identity hash code | 236 // Simulate a full heap so that generating an identity hash code |
| 173 // in subsequent calls will request GC. | 237 // in subsequent calls will request GC. |
| 174 SimulateFullSpace(CcTest::heap()->new_space()); | 238 SimulateFullSpace(CcTest::heap()->new_space()); |
| 175 SimulateFullSpace(CcTest::heap()->old_pointer_space()); | 239 SimulateFullSpace(CcTest::heap()->old_pointer_space()); |
| 176 | 240 |
| 177 // Calling Contains() should not cause GC ever. | 241 // Calling Contains() should not cause GC ever. |
| 178 int gc_count = isolate->heap()->gc_count(); | 242 int gc_count = isolate->heap()->gc_count(); |
| 179 CHECK(!table->Contains(*key)); | 243 CHECK(!table->Contains(*key)); |
| 180 CHECK(gc_count == isolate->heap()->gc_count()); | 244 CHECK(gc_count == isolate->heap()->gc_count()); |
| 181 | 245 |
| 182 // Calling Remove() will not cause GC in this case. | 246 // Calling Remove() will not cause GC in this case. |
| 183 table = ObjectHashSet::Remove(table, key); | 247 table = CloseTableSet::Remove(table, key); |
| 184 CHECK(gc_count == isolate->heap()->gc_count()); | 248 CHECK(gc_count == isolate->heap()->gc_count()); |
| 185 | 249 |
| 186 // Calling Add() should cause GC. | 250 // Calling Add() should cause GC. |
| 187 table = ObjectHashSet::Add(table, key); | 251 table = CloseTableSet::Add(table, key); |
| 188 CHECK(gc_count < isolate->heap()->gc_count()); | 252 CHECK(gc_count < isolate->heap()->gc_count()); |
| 189 } | 253 } |
| 190 #endif | 254 #endif |
| 191 | 255 |
| 192 | 256 |
| 193 #ifdef DEBUG | 257 #ifdef DEBUG |
| 194 TEST(ObjectHashTableCausesGC) { | 258 TEST(CloseTableMapCausesGC) { |
| 195 i::FLAG_stress_compaction = false; | 259 i::FLAG_stress_compaction = false; |
| 196 LocalContext context; | 260 LocalContext context; |
| 197 Isolate* isolate = CcTest::i_isolate(); | 261 Isolate* isolate = CcTest::i_isolate(); |
| 198 Factory* factory = isolate->factory(); | 262 Factory* factory = isolate->factory(); |
| 199 v8::HandleScope scope(context->GetIsolate()); | 263 v8::HandleScope scope(context->GetIsolate()); |
| 200 Handle<ObjectHashTable> table = factory->NewObjectHashTable(1); | 264 Handle<CloseTableMap> table = factory->NewCloseTableMap(); |
| 201 Handle<JSObject> key = factory->NewJSArray(0); | 265 Handle<JSObject> key = factory->NewJSArray(0); |
| 202 v8::Handle<v8::Object> key_obj = v8::Utils::ToLocal(key); | 266 v8::Handle<v8::Object> key_obj = v8::Utils::ToLocal(key); |
| 203 | 267 |
| 204 // Force allocation of hash table backing store for hidden properties. | 268 // Force allocation of hash table backing store for hidden properties. |
| 205 key_obj->SetHiddenValue(v8_str("key 1"), v8_str("val 1")); | 269 key_obj->SetHiddenValue(v8_str("key 1"), v8_str("val 1")); |
| 206 key_obj->SetHiddenValue(v8_str("key 2"), v8_str("val 2")); | 270 key_obj->SetHiddenValue(v8_str("key 2"), v8_str("val 2")); |
| 207 key_obj->SetHiddenValue(v8_str("key 3"), v8_str("val 3")); | 271 key_obj->SetHiddenValue(v8_str("key 3"), v8_str("val 3")); |
| 208 | 272 |
| 209 // Simulate a full heap so that generating an identity hash code | 273 // Simulate a full heap so that generating an identity hash code |
| 210 // in subsequent calls will request GC. | 274 // in subsequent calls will request GC. |
| 211 SimulateFullSpace(CcTest::heap()->new_space()); | 275 SimulateFullSpace(CcTest::heap()->new_space()); |
| 212 SimulateFullSpace(CcTest::heap()->old_pointer_space()); | 276 SimulateFullSpace(CcTest::heap()->old_pointer_space()); |
| 213 | 277 |
| 214 // Calling Lookup() should not cause GC ever. | 278 // Calling Lookup() should not cause GC ever. |
| 215 CHECK(table->Lookup(*key)->IsTheHole()); | 279 CHECK(table->Lookup(*key)->IsTheHole()); |
| 216 | 280 |
| 217 // Calling Put() should request GC by returning a failure. | 281 // Calling Put() should request GC by returning a failure. |
| 218 int gc_count = isolate->heap()->gc_count(); | 282 int gc_count = isolate->heap()->gc_count(); |
| 219 ObjectHashTable::Put(table, key, key); | 283 CloseTableMap::Put(table, key, key); |
| 220 CHECK(gc_count < isolate->heap()->gc_count()); | 284 CHECK(gc_count < isolate->heap()->gc_count()); |
| 221 } | 285 } |
| 222 #endif | 286 #endif |
| 287 |
| 288 } |
| OLD | NEW |