Use the LookupIterator for SetElement and friends

test/cctest/test-api.cc

Index: test/cctest/test-api.cc
diff --git a/test/cctest/test-api.cc b/test/cctest/test-api.cc
index 849833e58018e9714ff82228bacfdaf138461993..c0e83948d9d4fc7d839ccaf363986e7067b0238c 100644
--- a/test/cctest/test-api.cc
+++ b/test/cctest/test-api.cc
@@ -13266,12 +13266,10 @@ TEST(ForceSet) {
   CHECK_EQ(3, global->Get(access_property)->Int32Value());
   CHECK_EQ(1, force_set_set_count);
   CHECK_EQ(2, force_set_get_count);
-  // Forcing the property to be set should override the accessor without
-  // calling it
   global->ForceSet(access_property, v8::Int32::New(isolate, 8));

[Igor Sheludko, 2015/06/11 13:44:25]

I think it makes sense to add the above comment here.

[Toon Verwaest, 2015/06/11 14:26:50]

Done.

-  CHECK_EQ(8, global->Get(access_property)->Int32Value());
-  CHECK_EQ(1, force_set_set_count);
-  CHECK_EQ(2, force_set_get_count);
+  CHECK_EQ(3, global->Get(access_property)->Int32Value());
+  CHECK_EQ(2, force_set_set_count);
+  CHECK_EQ(3, force_set_get_count);
 }