TestRunner.js logs an expected subtest status of "OK" in a testStatus call when test URLs get out of sync here: http://hg.mozilla.org/mozilla-central/file/5327d2c8ee54/testing/mochitest/tests/SimpleTest/TestRunner.js#l691 "OK" isn't a valid subtest status, so an exception is thrown, see https://bugzilla.mozilla.org/show_bug.cgi?id=1050251#c9

"ERROR" also isn't a valid status. I wonder if this shouldn't be a testEnd message (as seen in bug 1051635), but maybe it's better to replace "OK" by "PASS" and "ERROR" with "FAIL" For now as the test could've been ended previously.

Comment on attachment 8473743 [details] [diff] [review] 0001-Bug-1054304-correct-test-status-for-tests-that-do-no.patch Review of attachment 8473743 [details] [diff] [review]: ----------------------------------------------------------------- Like we talked about a bit there's a pending issue of whether this really should be a test_end, but this will fix the invalid subtest status error in the meantime.

Thanks for the review!