Last Comment Bug 760080 - editor/libeditor/text/tests/test_bug569988.html ran additional tests after finish() was called
: editor/libeditor/text/tests/test_bug569988.html ran additional tests after fi...
Status: RESOLVED FIXED
:
Product: Core
Classification: Components
Component: Editor (show other bugs)
: Trunk
: All All
: -- trivial (vote)
: mozilla15
Assigned To: Mats Palmgren (:mats)
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2012-05-31 05:54 PDT by Mats Palmgren (:mats)
Modified: 2012-06-02 12:03 PDT (History)
2 users (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
Use info() for log messages that aren't tests. (1.71 KB, patch)
2012-05-31 05:58 PDT, Mats Palmgren (:mats)
ehsan: review-
Details | Diff | Review
Use info() for log messages that aren't tests. Move SimpleTest.finish() last. (2.06 KB, patch)
2012-05-31 09:08 PDT, Mats Palmgren (:mats)
ehsan: review+
Details | Diff | Review

Description Mats Palmgren (:mats) 2012-05-31 05:54:37 PDT
Fallout from the diagnostic patch in bug 677964.
Comment 1 Mats Palmgren (:mats) 2012-05-31 05:58:13 PDT
Created attachment 628693 [details] [diff] [review]
Use info() for log messages that aren't tests.
Comment 2 :Ehsan Akhgari (busy, don't ask for review please) 2012-05-31 08:38:36 PDT
Comment on attachment 628693 [details] [diff] [review]
Use info() for log messages that aren't tests.

SimpleTest.finish() should be the last thing that the test calls...
Comment 3 Mats Palmgren (:mats) 2012-05-31 09:08:11 PDT
Created attachment 628768 [details] [diff] [review]
Use info() for log messages that aren't tests.  Move SimpleTest.finish() last.
Comment 6 :Ehsan Akhgari (busy, don't ask for review please) 2012-06-02 12:03:06 PDT
https://hg.mozilla.org/mozilla-central/rev/1b230e42358e

Note You need to log in before you can comment on or make changes to this bug.