627624 - Make the crashtest for bug 428844 actually test stuff

Status: RESOLVED FIXED

(Core :: DOM: Editor, defect)

Description

[Boris Zbarsky [:bzbarsky]]

Right now it doesn't use reftest-wait, so ends the test before it reloads the subframe.

Comment 1

[(no longer active)]

Attachment: Patch (v1)

This is kind of tricky, as the page loaded inside the iframe isn't valid, and results in an error page, which doesn't trigger the load event.  I think the test intended to unload the iframe page instead of really reloading it.  This patch does that.

Comment 2

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 518905 [details] [diff] [review]
Patch (v1)

You don't need to remove the event listener:
   iframe.removeEventListener("load", arguments.callee, false);

Comment 3

[(no longer active)]

http://hg.mozilla.org/mozilla-central/rev/770b9488e967