New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add comprehensive coverage for WindowProperties object #27970
Add comprehensive coverage for WindowProperties object #27970
Conversation
I'll hold off on reviewing this until we settle whatwg/webidl#963; please ping for re-review if I don't get to it then! |
a89845b
to
b96e64b
Compare
@domenic Hey, I've decided not to pursue the spec changes and updated this PR to match the current spec. Although these tests were already approved by a WebKit reviewer, I was hoping you could take a look as well? |
html/browsers/the-window-object/named-access-on-the-window-object/window-named-properties.html
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Shall we close whatwg/webidl#963 and file bugs on Blink and Gecko?
Thank you for reviewing this Domenic! I've closed the issue and filed implementation bugs: |
var names = Object.getOwnPropertyNames(gsp); | ||
assert_equals(names.filter((name) => name == "baz").length, 1); | ||
|
||
assert_equals(gsp.baz, document.getElementsByTagName("iframe")[1]); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, this is wrong. #29063
Implementation bugs
This PR tests exclusively the current spec for named properties object.
Gecko (tested on Firefox Dev 89.0b14):
[[GetOwnProperty]]
doesn't return supported properties that are indices.Reflect.set
andReflect.defineProperty
incorrectly throwTypeError
in case of failure.[[OwnPropertyKeys]]
is not an ordinary method.Blink (tested on Chrome 92.0.4512.0):
馃帀 Now fixed! 馃帀[[DefineOwnProperty]]
doesn't fail for string keys, allowing expando properties.[[DefineOwnProperty]]
doesn't fail for symbols.Reflect.defineProperty
incorrectly returnstrue
for string properties (supported & not).[[Get]]
fails to perform named property visibility algorithm for supported indexed properties.[[Set]]
doesn't fail for symbols.Reflect.set
incorrectly returntrue
for string properties (supported & not).[[Delete]]
doesn't fail for symbols.WebKit: all passing with bug #222918.