Did you forget to ask for review? I don't understand why you make this todo(), it looks like it just doesn't need to be tested in production builds.
Created attachment 258811 [details] [diff] [review] updated, turning the todo() into an innocuous ok() I changed the todo() to an innocuous ok(). When the test is run on production builds, it definitely should pass.
Set the r? to sayrer. And I think setting isOK to false in the catch block and using the ternary operator instead of the if would be slightly better.
Whiteboard: [needs checkin] → [checkin needed]
Checking in test_bug351601.html; /cvsroot/mozilla/testing/mochitest/tests/test_bug351601.html,v <-- test_bug351601.html new revision: 1.2; previous revision: 1.1 done
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]
Component: Testing → DOM
QA Contact: testing → general
You need to log in before you can comment on or make changes to this bug.