Incorrect test description in fast/dom/Window/window-special-properties.html 
https://bugs.webkit.org/show_bug.cgi?id=23284
Summary Incorrect test description in fast/dom/Window/window-special-properties.html
Alexey Proskuryakov
Reported 2009-01-12 23:33:53 PST
The bug description says: "Our results match WinIE in all respects but one. We allow child frame names to take precedence over built-in and custom properties of the window object. This is the Mozilla behavior and is required to make some sites work because they use frame names that collide with built-in properties in Mozilla but not IE - specifically "toobar". See Radar 3870317." In <http://trac.webkit.org/changeset/31346>, test results were changed so that it is no longer true, without correcting the comment. It is also not clear to me whether the change to change the behavior was even intentional, and whether we broke rdar://3870317 as a result.
Attachments
Add attachment proposed patch, testcase, etc.
Ian 'Hixie' Hickson
Comment 1 2009-01-12 23:39:52 PST
Also of relevance, the equivalent test for HTMLDocument (named the same but s/window/document/) doesn't have that comment but _does_ have the described behavior, as IE does. Right now I'm speccing what IE does throughout, so if the Window test's comment is accurate (and the test is not) then it would be good to know so I can change it.
Sam Weinig
Comment 2 2009-01-13 00:30:21 PST
My guess is this was an oversight and not an intentional change, and that I just didn't read the comment at the top of the test before verifying that mozilla had the same behavior and calling it a day. In part, this is fallout from having two getOwnPropertySlot methods, one in JSDOMWindow and one in JSDOMWindowBase, that both implement part of the functionality. We should merge them and fix this.
Ian 'Hixie' Hickson
Comment 3 2009-01-13 00:38:13 PST
The spec now requires the Mozilla behavior, except that Mozilla doesn't currently support named getters on Window for anything _but_ frames.
Alexey Proskuryakov
Comment 4 2009-01-13 00:40:52 PST
We'll also need to extend the test to check for more than just window.location. The original radar and it duplicates are mostly about window.toolbar (very old versions of WebKit did not have this property, so some Safari-specific code broke when it was added).
Note You need to log in before you can comment on or make changes to this bug.