Summary: | postMessage should serialise dom objects to null by default | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Kim Grönholm <kim.1.gronholm> | ||||
Component: | New Bugs | Assignee: | Oliver Hunt <oliver> | ||||
Status: | ASSIGNED --- | ||||||
Severity: | Normal | CC: | hausmann, kim.1.gronholm, laszlo.gombos, oliver | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | All | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 39757 | ||||||
Attachments: |
|
Description
Kim Grönholm
2010-06-08 13:28:16 PDT
*** Bug 40232 has been marked as a duplicate of this bug. *** Created attachment 58911 [details]
Removes the test case with wrong expected output
Temporarily disabling the postMessage(window, "*") test as the expected
result and the behavior are both wrong. This doesn't fix the actual
issue but prevents the test case from failing due to any unrelated changes.
The test case should be enabled again with correct expected output after
the bug has been fixed.
From IRC discussion: - currently, the subtest raises "TypeError: Cannot post cyclic structures"; - with an unrelated patch, a different exception is raised, because the order of traversal changes; - the correct behavior is to return null. Comment on attachment 58911 [details]
Removes the test case with wrong expected output
It's best to keep this subtest to make sure that we don't break window serialization even more (e.g. that we don't start to crash).
I think that the best way to proceed is to change this subtest's results in the same unrelated change that will affect it.
|