Closed Bug 657554 Opened 10 years ago Closed 10 years ago

mochitest error output: test_bug519928.html, line 115: endTest is not defined

Categories

(Core :: DOM: Editor, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla6

People

(Reporter: dholbert, Assigned: dholbert)

References

()

Details

Attachments

(1 file)

Noticed this in mochitest output:
> JavaScript error: http://mochi.test:8888/tests/editor/composer/test/test_bug519928.html, line 115: endTest is not defined

Indeed, test_bug519928.html calls "endTest()" which is never defined.

This happens right after a call to SimpleTest.finish(), so I think we're fine to just finish with that and drop the "endTest()" call.
OS: Linux → All
Hardware: x86_64 → All
Attached patch fix: delete lineSplinter Review
Assignee: nobody → dholbert
Status: NEW → ASSIGNED
Attachment #532880 - Flags: review?(ehsan)
Comment on attachment 532880 [details] [diff] [review]
fix: delete line

Oh, I should have caught this when reviewing bug 579303.  :/  Thanks for finding and fixing this.
Attachment #532880 - Flags: review?(ehsan) → review+
Pushed:
http://hg.mozilla.org/mozilla-central/rev/1d5ffdf05dab
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Whiteboard: fixed-in-cedar
Target Milestone: --- → mozilla6
You need to log in before you can comment on or make changes to this bug.