1129153 - Change [[Delete]] to follow ES6

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Comment 1

[Tom S [:evilpie]]

Attachment: v1 - Implement ES6 [[Delete]]

This was already pretty much okay, we were just waling the proto chain with LookupProperty.

Comment 2

[Jeff Walden [:Waldo]]

Comment on attachment 8558805 [details] [diff] [review]
v1 - Implement ES6 [[Delete]]

Review of attachment 8558805 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/jit-test/tests/proxy/delete-not-invoked-on-proto.js
@@ +1,2 @@
> +// Create Proxy that throws for everything.
> +var {proxy, revoke} = Proxy.revocable({}, {});

Instead of {} for handler, triggering default-er behavior (right?), seems better to use

var angryHandler = new Proxy({}, {
    get(t, id) { throw new Error("angryHandler should not be queried (" + id + ")"); }
});

as handler.  (Props to someone for putting that in a test I remember reviewing, tho I'm not sure if it was the one I just cribbed that from.)

::: js/src/vm/NativeObject.cpp
@@ +2232,5 @@
> +
> +    // Step 4.
> +    if (!shape) {
> +        // If no property call the class's delProperty hook, passing succeeded
> +        // as the result parameter. This always succeedes when there is no hook.

succeeds

@@ +2236,5 @@
> +        // as the result parameter. This always succeedes when there is no hook.
> +        return CallJSDeletePropertyOp(cx, obj->getClass()->delProperty, obj, id, succeeded);
> +    }
> +
> +    cx->runtime()->gc.poke();

wheeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeee

Comment 3

[Tom S [:evilpie]]

Comment on attachment 8558805 [details] [diff] [review]
v1 - Implement ES6 [[Delete]]

Review of attachment 8558805 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/jit-test/tests/proxy/delete-not-invoked-on-proto.js
@@ +1,2 @@
> +// Create Proxy that throws for everything.
> +var {proxy, revoke} = Proxy.revocable({}, {});

Well the idea that a revoked proxy throws for every hook, so I don't need to do it manually.

Comment 4

[Tom S [:evilpie]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/ed5c9eded5a1

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/ed5c9eded5a1