[webkit-reviews] review granted: [Bug 24339] Add test for potential crash in window.frames.length : [Attachment 28289] New test + result, no form submission

[bugzilla-daemon at webkit.org]

Alexey Proskuryakov <ap at webkit.org> has granted Pam Greene <pam at chromium.org>'s
request for review:
Bug 24339: Add test for potential crash in window.frames.length
https://bugs.webkit.org/show_bug.cgi?id=24339

Attachment 28289: New test + result, no form submission
https://bugs.webkit.org/attachment.cgi?id=28289&action=review

------- Additional Comments from Alexey Proskuryakov <ap at webkit.org>
> +<iframe id="subframe"><p id="contents">Subframe</p></iframe>

This paragraph will be ignored - did you intend to do src='data:text/html,<p
id="contents">Subframe</p>'? But in this case, the test would probably need to
run from an onload handler, as data: URL loading is async.

r=me either way.