Closed Bug 725942 Opened 8 years ago Closed 8 years ago

mochitest-browser-chrome: browser-test.js should ignore empty console messages

Categories

(Testing :: Mochitest, defect, P4, minor)

defect

Tracking

(firefox10 wontfix, firefox11 wontfix, firefox12 wontfix, firefox-esr10 wontfix)

VERIFIED FIXED
mozilla13
Tracking Status
firefox10 --- wontfix
firefox11 --- wontfix
firefox12 --- wontfix
firefox-esr10 --- wontfix

People

(Reporter: sgautherie, Assigned: sgautherie)

References

()

Details

Attachments

(1 file)

While working on SeaMonkey bug 521305, I noticed 2 empty console messages reported by that test.

I ran a Firefox Try build to confirm:
https://tbpl.mozilla.org/?tree=Try&rev=a872216b6fef
90+ empty messages detected

Searching Bugzilla, I find related bug 282253 and bug 564962.
It seems this behavior is +/- a feature and that empty messages should be ignored.
Flags: in-testsuite-
Attachment #595993 - Flags: review? → review?(dtownsend+bugmail)
Attachment #595993 - Flags: review?(dtownsend+bugmail) → review+
Comment on attachment 595993 [details] [diff] [review]
(Av1) browser-test.js: Ignore empty console messages
[Checked in: Comment 2]

https://hg.mozilla.org/mozilla-central/rev/ad0038e0af1f
Attachment #595993 - Attachment description: (Av1) browser-test.js: Ignore empty console messages → (Av1) browser-test.js: Ignore empty console messages [Checked in: Comment 2]
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
https://tbpl.mozilla.org/php/getParsedLog.php?id=9337216&tree=Firefox&full=1
Rev3 Fedora 12 mozilla-central opt test mochitest-other on 2012-02-14 14:13:58 PST for push ad0038e0af1f

No more
"TEST-INFO | chrome://mochitests/content/browser/browser/base/content/test/browser_sanitizeDialog.js | Console message:"
as an example.

V.Fixed
Status: RESOLVED → VERIFIED
Component: BrowserTest → Mochitest
You need to log in before you can comment on or make changes to this bug.