Revert "Make Function.length and Function.name configurable properties."

src/runtime.cc

Index: src/runtime.cc
diff --git a/src/runtime.cc b/src/runtime.cc
index d2727db79bb232d78bcdc7e8ebc80d109548362b..3596add429ee9335994c525ad946be41648d6842 100644
--- a/src/runtime.cc
+++ b/src/runtime.cc
@@ -5068,14 +5068,11 @@ RUNTIME_FUNCTION(MaybeObject*, Runtime_DefineOrRedefineDataProperty) {
     if (callback->IsAccessorInfo()) {
       return isolate->heap()->undefined_value();
     }
-    // Provided a read-only property isn't being reconfigured, avoid redefining
-    // foreign callback as data property, just use the stored setter to update
-    // the value instead.
+    // Avoid redefining foreign callback as data property, just use the stored
+    // setter to update the value instead.
     // TODO(mstarzinger): So far this only works if property attributes don't
     // change, this should be fixed once we cleanup the underlying code.
-    if (callback->IsForeign() &&
-        lookup.GetAttributes() == attr &&
-        !(attr & READ_ONLY)) {
+    if (callback->IsForeign() && lookup.GetAttributes() == attr) {
       Handle<Object> result_object =
           JSObject::SetPropertyWithCallback(js_object,
                                             callback,