Chromium Code Reviews| Index: src/builtins/builtins-object.cc |
| diff --git a/src/builtins/builtins-object.cc b/src/builtins/builtins-object.cc |
| index 8be615013c9ee8b17739790f3797e975361eff43..1c1b4dfa04757fb9302510cd0b4d62ce25a1c48d 100644 |
| --- a/src/builtins/builtins-object.cc |
| +++ b/src/builtins/builtins-object.cc |
| @@ -114,7 +114,7 @@ Object* ObjectDefineAccessor(Isolate* isolate, Handle<Object> object, |
| // To preserve legacy behavior, we ignore errors silently rather than |
| // throwing an exception. |
| Maybe<bool> success = JSReceiver::DefineOwnProperty( |
| - isolate, receiver, name, &desc, Object::DONT_THROW); |
| + isolate, receiver, name, &desc, Object::DONT_THROW, false); |
|
Franzi
2017/05/10 09:17:52
I'd prefer an enum instead of a bool. "false" is j
|
| MAYBE_RETURN(success, isolate->heap()->exception()); |
| if (!success.FromJust()) { |
| isolate->CountUsage(v8::Isolate::kDefineGetterOrSetterWouldThrow); |