549117 - [SeaMonkey] mochitest-plain: new test_bug518122.html is perma-orange, "Wrong value ..."

Status: VERIFIED FIXED

(Core :: DOM: Events, defect)

Description

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1267275768.1267277641.6140.gz&fulltext=1
Linux comm-central-trunk debug test mochitests on 2010/02/27 05:02:48
{
39409 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug518122.html | Wrong value (wrap=null, simple_test=6) - got "foo", expected " foo"
39411 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug518122.html | Wrong value (wrap=null, simple_test=8) - got "foobar", expected "foo bar"
39412 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug518122.html | Wrong value (wrap=null, simple_test=9) - got "foo\nbar", expected "foo bar"
39414 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug518122.html | Wrong value (wrap=null, simple_test=11) - got "foo", expected " foo"
39416 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug518122.html | Wrong value (wrap=null, simple_test=13) - got "foobar", expected "foo bar"
39417 ERROR TEST-UNEXPECTED-FAIL | /tests/content/html/content/test/test_bug518122.html | Wrong value (wrap=null, simple_test=14) - got "foo\nbar", expected "foo bar"
}

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1267275418.1267288402.7110.gz&fulltext=1
OS X 10.5 comm-central-trunk debug test mochitests on 2010/02/27 04:56:58
too

Comment 1

[Olli Pettay [:smaug][[e-mail]]]

So did some other patch cause this?
Does this happen on Seamonkey-tinderbox only?

Comment 2

[Olli Pettay [:smaug][[e-mail]]]

..because bug 518122 was pushed few weeks ago, and if this didn't happen then,
this shouldn't block bug 518122.

Comment 3

[Serge Gautherie (:sgautherie)]

(In reply to comment #1)

> So did some other patch cause this?

No idea.

> Does this happen on Seamonkey-tinderbox only?

Yes, afaik.

(In reply to comment #2)
> ..because bug 518122 was pushed few weeks ago, and if this didn't happen then,
> this shouldn't block bug 518122.

But SeaMonkey trunk wasn't running this test suite at that time:
http://tinderbox.mozilla.org/showbuilds.cgi?tree=SeaMonkey&maxdate=1265600000&hours=24&legend=0&norules=1

Comment 4

[Boris Zbarsky [:bzbarsky]]

Bug 549094 comment 1 explains what's going on here...

Comment 5

[[e-mail]]

Although presumably in this case we can set the preference to what we want?

Comment 6

[(no longer active)]

Attachment: Patch (v1)

Set the pref value to what we expect.

Comment 7

[(no longer active)]

Attachment: Patch (v1.1)

Actually, let's not use magic numbers here!

Comment 8

[Boris Zbarsky [:bzbarsky]]

Might be safer to use mozprefs.js pref() here (the version that takes a function to run).

Comment 9

[Boris Zbarsky [:bzbarsky]]

And again, the DOM bug this is showing is quite real.  Inconsistent DOM behavior on simple value sets is .... bad.

Comment 10

[Serge Gautherie (:sgautherie)]

V.Fixed, per FF and SM tinderboxes.