PerformanceObserver.supportedEntryTypescan't be annotated with [SameObject]

[petervanderbeken]

The supportedEntryTypes attribute has a FrozenArray type. [SameObject] is only valid if the attribute's type is an interface or object, see https://webidl.spec.whatwg.org/#SameObject:

The [SameObject] extended attribute must not be used on anything other than a read only attribute whose type is an interface type or object.

[clelland]

@yoavweiss who may have some historical context here.

[yoavweiss]

I vaguely remember the current state being a result of discussions on #117

I don't personally have a strong opinion on this.

[clelland]

whatwg/webidl#1402 has some recent movement on this issue.

I don't expect that we actually need this to be [SameObject]; I don't think people are modifying it and expecting it to retain those modifications. This was likely to avoid recalculating or allocating more memory every time the accessor is called.

[clelland]

We also don't technically need FrozenArray here either. I think that what we want is to return an array, and not recalculate that array every time it's requested. Perhaps as Domenic alludes to in whatwg/webidl#1402 (comment), we can avoid being blocked on whatwg/webidl#1402 if we change the spec text to just do what we want in the getter.

[tunetheweb]

It would be lovely if it was an array that could be updated but didn't need to be recalculated each time. So enabling things via origin trials for example could update these even if it's already been read. Ran into this with LoAF when it was enabled after first being read for another reason.

[yoavweiss]

Changing the values of the array over time would be a web-exposed change, although I doubt that it'd be something anyone would notice..

[clelland]

1. We should remove [SameObject], and just enforce that in the processing model. Can send a PR for that.
2. To handle origin trials, we might want to switch from a FrozenArray to a regular sequence. That's less clear though.