Serge Gautherie (:sgautherie) Assignee
	Description • 16 years ago

At least, I'd like the summary formats to be synchronized.

browser-harness.xul: (BrowserChrome)
{
Browser Chrome Test Summary
	Pass: 1272
	Fail: 0
	Todo: 5
}

SimpleTest.js: (MochiTest, MochiChrome, A11y)
{
1327 INFO Passed: 1177
1328 INFO Failed: 0
1329 INFO Todo:   79
}

	Serge Gautherie (:sgautherie) Assignee
	Comment 1 • 16 years ago

Bug 364043 added this to new console output but not to existing file output.

This difference broke my bug 473259 patch Ev1,
because I test locally from the file, whereas buildbot uses the console :-/

	Serge Gautherie (:sgautherie) Assignee
	Comment 2 • 16 years ago

Comment on attachment 379695 [details] [diff] [review]
(Av1) Mochitest: remove '*** ' console output
[Checkin: Comment 2 & 3]


http://hg.mozilla.org/mozilla-central/rev/cf17e49a665a

	Serge Gautherie (:sgautherie) Assignee
	Comment 3 • 16 years ago

Comment on attachment 379695 [details] [diff] [review]
(Av1) Mochitest: remove '*** ' console output
[Checkin: Comment 2 & 3]


http://hg.mozilla.org/releases/mozilla-1.9.1/rev/a973204ab988

	Serge Gautherie (:sgautherie) Assignee
	Comment 4 • 16 years ago

Bug 387455 added this to text output but it differs from html output,
and more importantly from other Mochitest output(s).

This has always needed a specific case in buildbot: see bug 473259 patch Ev1a.
(That buildbot file will need to be updated in sync' too...)

	Serge Gautherie (:sgautherie) Assignee
	Comment 5 • 15 years ago

rcampbell, ping for review ?

	Serge Gautherie (:sgautherie) Assignee
	Comment 6 • 15 years ago

rcampbell: ping for review?

	Serge Gautherie (:sgautherie) Assignee
	Comment 7 • 15 years ago

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/mozilla-central/rev/b0c89755c903
Bv1, unbitrotted.

	Serge Gautherie (:sgautherie) Assignee
	Comment 8 • 15 years ago

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/releases/mozilla-1.9.2/rev/eaea03ee6784

	Serge Gautherie (:sgautherie) Assignee
	Comment 9 • 15 years ago

I backed out patch Bv1 because I missed an additional change in buildbotcustom.
Will reland this patch when bbc is ready.

	Serge Gautherie (:sgautherie) Assignee
	Comment 10 • 15 years ago

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/mozilla-central/rev/1d95bcb9b00f

	Mike Beltzner [:beltzner, not reading bugmail]
	Comment 11 • 15 years ago

Can we back this out? Since it landed there have been a ton of orange boxes.

	:Gijs (he/him)
	Comment 12 • 15 years ago

(In reply to comment #11)
> Can we back this out? Since it landed there have been a ton of orange boxes.

I backed this out per https://bugzilla.mozilla.org/show_bug.cgi?id=473259#c55, where Serge says it needs to be backed out because of an earlier backout.

http://hg.mozilla.org/mozilla-central/rev/2bf3249fe4bb

	Serge Gautherie (:sgautherie) Assignee
	Comment 13 • 15 years ago

(In reply to comment #11)
> Since it landed there have been a ton of orange boxes.

No, there was only one reported,
the other reports were bug 473259 comment 55 "mess".

(In reply to comment #12)
> I backed this out per https://bugzilla.mozilla.org/show_bug.cgi?id=473259#c55,

Bug 473259 patch was re-landed after that.
Yet, I'll let the situation settle for a while before re-landing this patch...

	Serge Gautherie (:sgautherie) Assignee
	Comment 14 • 15 years ago

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/mozilla-central/rev/7f6e2f4abdfd

	Serge Gautherie (:sgautherie) Assignee
	Comment 15 • 15 years ago

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/releases/mozilla-1.9.2/rev/58087da8caa6

	Serge Gautherie (:sgautherie) Assignee
	Comment 16 • 15 years ago

Comment on attachment 379786 [details] [diff] [review]
(Bv1) BrowserChrome: always use 'Passed' and 'Failed' idents
[Checkin: See comment 7+14 & 15 & 16]


http://hg.mozilla.org/releases/mozilla-1.9.1/rev/5e85a1d1c673