Closed Bug 469593 Opened 12 years ago Closed 12 years ago

test_423060.xul fails on SeaMonkey (though the error is reported on test_428653.xul); and test_autocomplete2.xul can fail/leak too

Categories

(Firefox Graveyard :: RSS Discovery and Preview, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Firefox 3.1b3

People

(Reporter: kairo, Assigned: kairo)

References

(Blocks 1 open bug)

Details

(Keywords: verified1.9.1)

Attachments

(1 file)

When SeaMonkey switched to using places, we discovered we suddenly had turned on test_423060.xul, which is a large structural mess in form of a testcase:
It tests a toolkit/components/feeds fix by using a browser/components/feeds feature and is living in toolkit/components/places - probably because it uses some other files that conveniently happened to already sit in places tests.

It must have been clear that this calls for problems - for one, in any config that uses places but not feeds, but even more, in any config that uses toolkit/ but not browser/ (which is the case SeaMonkey is running into here, as nothing says that chrome mochitests shouldn't be run with anything else than Firefox).

I see two options to clean this up:
1) move it to browser/components/feeds as it's heavily using browser-only code (which I'd prefer)
2) move it to toolkit/components/feeds as it tests a toolkit fix, and ifdef it out for SeaMonkey as long as that one doesn't provide web feed handlers

In both cases, we probably need to copy the demohandler and sample feed from places to the respective location.

I'm willing to do some work on this to get SeaMonkey fixed, but I'd need some advice on which direction to go.
Blocks: SmTestFail
Flags: wanted-firefox3.1?
This patch moves this patch over into browser/components/feeds where the web feed handler stuff lives that is used as the base for the test.
Assignee: nobody → kairo
Status: NEW → ASSIGNED
Attachment #353898 - Flags: review?(gavin.sharp)
Attachment #353898 - Attachment description: miove test to browser/components/feeds → move test to browser/components/feeds
Attachment #353898 - Flags: review?(gavin.sharp) → review+
Thanks, pushed as http://hg.mozilla.org/mozilla-central/rev/35de15b279bc and http://hg.mozilla.org/releases/mozilla-1.9.1/rev/9255910ce8f5
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
verified1.9.1, as the boxes do not report any of these errors
anymore.

***

Ftr, see
*bug 468341 comment 22 for the exact error message.
 *afawk, intermittent test_autocomplete2.xul failure was only a side effect of this bug.
*bug 468341 comment 24 for why this bug error is reported on the next test.
Flags: wanted-firefox3.1?
Keywords: verified1.9.1
Summary: test_423060.xul fails on SeaMonkey → test_423060.xul fails on SeaMonkey (though the error is reported on test_428653.xul); and test_autocomplete2.xul can fail/leak too
Target Milestone: --- → Firefox 3.1b3
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20081230 SeaMonkey/2.0a3pre] (home, optim default) (W2Ksp4)
(http://hg.mozilla.org/mozilla-central/rev/6201c0669e15
 +http://hg.mozilla.org/comm-central/rev/2854a16a867f)

V.Fixed
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.