Remove ForceDelete

test/cctest/test-object-observe.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 310 matching lines @@
   obj->Set(String::NewFromUtf8(v8_isolate, "foo"),
            Number::New(v8_isolate, 7));
   obj->Set(1, Number::New(v8_isolate, 2));
   // ForceSet should work just as well as Set
   obj->ForceSet(String::NewFromUtf8(v8_isolate, "foo"),
                 Number::New(v8_isolate, 3));
   obj->ForceSet(Number::New(v8_isolate, 1), Number::New(v8_isolate, 4));
   // Setting an indexed element via the property setting method
   obj->Set(Number::New(v8_isolate, 1), Number::New(v8_isolate, 5));
   // Setting with a non-String, non-uint32 key
-  obj->ForceSet(Number::New(v8_isolate, 1.1), Number::New(v8_isolate, 6),
-                DontDelete);
+  obj->Set(Number::New(v8_isolate, 1.1), Number::New(v8_isolate, 6));
   obj->Delete(String::NewFromUtf8(v8_isolate, "foo"));
   obj->Delete(1);
-  obj->ForceDelete(Number::New(v8_isolate, 1.1));
+  obj->Delete(Number::New(v8_isolate, 1.1));
 
   // Force delivery
   // TODO(adamk): Should the above set methods trigger delivery themselves?
   CompileRun("void 0");
   CHECK_EQ(9, CompileRun("records.length")->Int32Value());
   const RecordExpectation expected_records[] = {
     { obj, "add", "foo", Handle<Value>() },
     { obj, "add", "1", Handle<Value>() },
     // Note: use 7 not 1 below, as the latter triggers a nifty VS10 compiler bug
     // where instead of 1.0, a garbage value would be passed into Number::New.
@@ skipping 482 matching lines @@
                       &AccessorSetter);
   context->Global()->Set(String::NewFromUtf8(isolate, "obj"), object);
   CompileRun(
       "var records = null;"
       "Object.observe(obj, function(r) { records = r });"
       "obj.accessor = 43;");
   CHECK(CompileRun("records")->IsNull());
   CompileRun("Object.defineProperty(obj, 'accessor', { value: 44 });");
   CHECK(CompileRun("records")->IsNull());
 }