Allow setting accessor infos over read-only but configurable properties.
authorverwaest <verwaest@chromium.org>
Tue, 14 Jul 2015 17:43:09 +0000 (10:43 -0700)
committerCommit bot <commit-bot@chromium.org>
Tue, 14 Jul 2015 17:43:15 +0000 (17:43 +0000)
BUG=

Review URL: https://codereview.chromium.org/1228373004

Cr-Commit-Position: refs/heads/master@{#29655}

src/objects.cc

index eb11561..aeb6bb5 100644 (file)
@@ -6338,7 +6338,7 @@ MaybeHandle<Object> JSObject::SetAccessor(Handle<JSObject> object,
 
   // ES5 forbids turning a property into an accessor if it's not
   // configurable. See 8.6.1 (Table 5).
-  if (it.IsFound() && (it.IsReadOnly() || !it.IsConfigurable())) {
+  if (it.IsFound() && !it.IsConfigurable()) {
     return it.factory()->undefined_value();
   }