Closed
Bug 725942
Opened 12 years ago
Closed 12 years ago
mochitest-browser-chrome: browser-test.js should ignore empty console messages
Categories
(Testing :: Mochitest, defect, P4)
Testing
Mochitest
Tracking
(firefox10 wontfix, firefox11 wontfix, firefox12 wontfix, firefox-esr10 wontfix)
VERIFIED
FIXED
mozilla13
People
(Reporter: sgautherie, Assigned: sgautherie)
References
()
Details
Attachments
(1 file)
|
987 bytes,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
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-
|
Assignee | |
Comment 1•12 years ago
|
||
Attachment #595993 -
Flags: review?
|
|
Assignee | |
Updated•12 years ago
|
Attachment #595993 -
Flags: review? → review?(dtownsend+bugmail)
|
||
Updated•12 years ago
|
Attachment #595993 -
Flags: review?(dtownsend+bugmail) → review+
|
|
Assignee | |
Comment 2•12 years ago
|
||
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]
|
|
Assignee | |
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 3•12 years ago
|
||
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
status-firefox-esr10:
--- → affected
status-firefox10:
--- → wontfix
status-firefox11:
--- → wontfix
status-firefox12:
--- → wontfix
|
||
Updated•6 years ago
|
Component: BrowserTest → Mochitest
|
||
Updated•5 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•