Per-docs, this should be allowed but the code doesn't check the result of the property handler. Currently this can only be done via throwing an exception from within the delProperty handler.
Created attachment 531839 [details] [diff] [review]
A quick scan via mxr shows only two custom delete property hooks and neither sets *vp.
Comment on attachment 531839 [details] [diff] [review]
I thought I plus'd this a long time ago.
I note that this leaves a delProperty call in Object.defineProperty (DefinePropertyOnObject, specifically) unchanged. I think that's OK. We're really after V8 API parity here.
Rob, did we not need this after all? It hasn't landed. Checkin-needed?
checkin-needed - I haven't found the time to pull a t-m tree and push (or actually figure out where it should land now).