Make the crashtest for bug 428844 actually test stuff

RESOLVED FIXED

Status

()

Core
Editor
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: bz, Assigned: Away for a while)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

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

Comment 1

7 years ago
Created attachment 518905 [details] [diff] [review]
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.
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #518905 - Flags: review?(roc)
Comment on attachment 518905 [details] [diff] [review]
Patch (v1)

You don't need to remove the event listener:
   iframe.removeEventListener("load", arguments.callee, false);
Attachment #518905 - Flags: review?(roc) → review+
(Assignee)

Comment 3

7 years ago
http://hg.mozilla.org/mozilla-central/rev/770b9488e967
Status: ASSIGNED → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.