Split Put into Put and Remove

test/cctest/test-ordered-hash-table.cc

Index: test/cctest/test-ordered-hash-table.cc
diff --git a/test/cctest/test-ordered-hash-table.cc b/test/cctest/test-ordered-hash-table.cc
index 1614e4b27e208afdbc5fdfe923f853852c7cf610..b9f7606e7ca175e2a885ca49a24d8dd76f803709 100644
--- a/test/cctest/test-ordered-hash-table.cc
+++ b/test/cctest/test-ordered-hash-table.cc
@@ -176,8 +176,7 @@ TEST(Map) {
   ordered_map = OrderedHashMap::Put(ordered_map, obj, val);
   CHECK_EQ(1, ordered_map->NumberOfElements());
   CHECK(ordered_map->Lookup(obj)->SameValue(*val));
-  ordered_map = OrderedHashMap::Put(
-      ordered_map, obj, factory->the_hole_value());
+  ordered_map = OrderedHashMap::Remove(ordered_map, obj);

[adamk, 2014/05/30 20:13:58]

Probably want to test the was_present version as well...

[arv (Not doing code reviews), 2014/05/30 20:53:07]

Done.

   CHECK_EQ(0, ordered_map->NumberOfElements());
   CHECK(ordered_map->Lookup(obj)->IsTheHole());
 
@@ -238,14 +237,10 @@ TEST(Map) {
                         true);
 
   // Test shrinking
-  ordered_map = OrderedHashMap::Put(
-      ordered_map, obj, factory->the_hole_value());
-  ordered_map = OrderedHashMap::Put(
-      ordered_map, obj1, factory->the_hole_value());
-  ordered_map = OrderedHashMap::Put(
-      ordered_map, obj2, factory->the_hole_value());
-  ordered_map = OrderedHashMap::Put(
-      ordered_map, obj3, factory->the_hole_value());
+  ordered_map = OrderedHashMap::Remove(ordered_map, obj);
+  ordered_map = OrderedHashMap::Remove(ordered_map, obj1);
+  ordered_map = OrderedHashMap::Remove(ordered_map, obj2);
+  ordered_map = OrderedHashMap::Remove(ordered_map, obj3);
   CHECK_EQ(1, ordered_map->NumberOfElements());
   CHECK_EQ(2, ordered_map->NumberOfBuckets());
 }