Chromium Code Reviews| Index: src/runtime/runtime-classes.cc |
| diff --git a/src/runtime/runtime-classes.cc b/src/runtime/runtime-classes.cc |
| index ca2df80355b741f9ef30a81ec284070705376e2e..16edc99e57098f766a8776746f0f8df19dcbef10 100644 |
| --- a/src/runtime/runtime-classes.cc |
| +++ b/src/runtime/runtime-classes.cc |
| @@ -167,13 +167,6 @@ static MaybeHandle<Object> DefineClass(Isolate* isolate, |
| prototype, attribs), |
| Object); |
| - // TODO(arv): Only do this conditionally. |
|
arv (Not doing code reviews)
2017/01/05 05:18:43
Thanks
|
| - Handle<Symbol> home_object_symbol(isolate->heap()->home_object_symbol()); |
| - RETURN_ON_EXCEPTION( |
| - isolate, JSObject::SetOwnPropertyIgnoreAttributes( |
| - constructor, home_object_symbol, prototype, DONT_ENUM), |
| - Object); |
| - |
| if (!constructor_parent.is_null()) { |
| MAYBE_RETURN_NULL(JSObject::SetPrototype(constructor, constructor_parent, |
| false, Object::THROW_ON_ERROR)); |
| @@ -195,7 +188,8 @@ static MaybeHandle<Object> DefineClass(Isolate* isolate, |
| handle(Smi::FromInt(end_position), isolate), STRICT), |
| Object); |
| - return constructor; |
| + // Caller already has access to constructor, so return the prototype. |
| + return prototype; |
| } |