Revert of Reland "Use C++ implementation of Object.definePropert{y,ies}"

test/cctest/test-api.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 8709 matching lines @@
 
   CompileRun("Object.seal(other)");
   ExpectTrue("Object.isExtensible(other)");
 
   // Regression test for issue 1250.
   // Make sure that we can set the accessible accessors value using normal
   // assignment.
   CompileRun("other.accessible_prop = 42");
   CHECK_EQ(42, g_echo_value);
 
-  // [[DefineOwnProperty]] always throws for access-checked objects.
-  CHECK(
-      CompileRun("Object.defineProperty(other, 'accessible_prop', {value: 43})")
-          .IsEmpty());
-  CHECK(CompileRun("other.accessible_prop == 42")->IsTrue());
-  CHECK_EQ(42, g_echo_value);  // Make sure we didn't call the setter.
+  v8::Handle<Value> value;
+  CompileRun("Object.defineProperty(other, 'accessible_prop', {value: -1})");
+  value = CompileRun("other.accessible_prop == 42");
+  CHECK(value->IsTrue());
 }
 
 
 static bool AccessAlwaysBlocked(Local<v8::Object> global, Local<Value> name,
                                 v8::AccessType type, Local<Value> data) {
   i::PrintF("Access blocked.\n");
   return false;
 }
 
 
@@ skipping 13170 matching lines @@
   ExpectTrue("object[Symbol.isConcatSpreadable]");
 
   // If access check fails, the value of @@isConcatSpreadable is ignored
   allowed_access = false;
   CompileRun("var result = [].concat(object)");
   ExpectTrue("Array.isArray(result)");
   ExpectTrue("result[0] === object");
   ExpectTrue("result.length === 1");
   ExpectTrue("object[Symbol.isConcatSpreadable] === undefined");
 }