eshost icon indicating copy to clipboard operation
eshost copied to clipboard

Setting a realm's global object

Open jugglinmike opened this issue 10 years ago • 1 comments

eshost currently allows for the specification of global bindings during ECMAScript realm creation. These bindings are created as enumerable, writable, and configurable properties of the global object.

The ES2015 specification defines behavior for global object properties with different descriptors. Although some of these can be tested by runtime creation of properties, e.g.

Object.defineProperty(this, 'foo', { value: 0, writable: false });
foo++;
assert.sameValue(foo, 0);

...others cannot. For example the semantics of GlobalDeclarationEvaluation dictate (via CanDeclareGlobalFunction) that if the global object has a property named foo that has the value undefined and the descriptor {[[Configurable]]: false, [[Writable]]: true, [[Enumerable]]: true}, the following code should execute without error:

foo();
function foo() {}

One solution would be to extend the eshost API to respect the property descriptors of the provided object, but the spec also accounts for global objects that are exotic. For example, here's a non-normative note from GlobalDeclarationEvaluation:

13- NOTE: No abnormal terminations occur after this algorithm step if the global object is an ordinary object. However, if the global object is a Proxy exotic object it may exhibit behaviours that cause abnormal terminations in some of the following steps.

So I think we need the ability to specify the global object directly. Does that sound right to you, @bterlson? I'm also interested in this detail as it relates to Test262. If what I've written above is accurate, we will one day need to write tests that dictate a realm's global object value.

jugglinmike avatar Mar 10 '16 17:03 jugglinmike

I may be putting the horse before the carriage here. I've opened https://github.com/tc39/test262/issues/538 to learn more about exotic global objects.

jugglinmike avatar Mar 11 '16 15:03 jugglinmike