458047 - 456147.xul reftest (still) fails on MacOSX

Status: RESOLVED FIXED

(Core :: XUL, defect)

Description

[Serge Gautherie (:sgautherie)]

http://tinderbox.mozilla.org/showlog.cgi?log=SeaMonkey/1222844241.1222849852.6908.gz
MacOSX 10.4 comm-central dep unit test on 2008/09/30 23:57:21
{
REFTEST TEST-UNEXPECTED-FAIL | file:///builds/slave/comm-central-macosx/build/mozilla/layout/reftests/bugs/456147.xul | 

*** 33216 ERROR TEST-UNEXPECTED-FAIL | /tests/dom/tests/mochitest/general/test_offsets.html | scrollbox scrollHeight - got 125, expected 126
}

I don't know if the two failures are related, on the started on the same build.

Comment 1

[Serge Gautherie (:sgautherie)]

Ah, Firefox "MacOSX Darwin 9.2.2" test boxes have it too.

Comment 2

[Markus Stange [:mstange]]

I'm currently looking for a regression range - the vertical alignment is right in yesterday's nightly (the strikethrough-colors obviously not) but wrong in a build of the revision immediately before bug 456147 landed.

Comment 3

[Markus Stange [:mstange]]

(In reply to comment #2)
> bug wrong in a
> build of the revision immediately before bug 456147 landed.

That's not correct. I'll report again when I find out more.

Comment 4

[Serge Gautherie (:sgautherie)]

(In reply to comment #0)
> *** 33216 ERROR TEST-UNEXPECTED-FAIL |
> /tests/dom/tests/mochitest/general/test_offsets.html | scrollbox scrollHeight -
> got 125, expected 126
> }
> 
> I don't know if the two failures are related, on the started on the same build.

This one is (now) bug 457194 comment 8 ;-)

Comment 5

[Markus Stange [:mstange]]

> I'm currently looking for a regression range - the vertical alignment is right
> in yesterday's nightly

It looks like I was being stupid here. It's wrong in yesterday's nightly, too. So this isn't a regression.

I don't know what causes the wrong vertical align. But since it only happens on Mac OS X, it's probably OK to keep this as a known failure.

Comment 6

[Serge Gautherie (:sgautherie)]

For now, this was worked around by:

http://hg.mozilla.org/mozilla-central/rev/74d507e4ca41
Mark reftest for bug 456147 as failing on Mac; problem isn't caused by this bug but by an unknown earlier regression

Comment 7

[Markus Stange [:mstange]]

I was mistaken; there was no "earlier regression".
Whether the test passes on Mac OS X apparently depends on the theme. When I use the GrApple theme (http://www.takebacktheweb.org/), the test passes; with the default theme it fails.
I have no idea what's going on.

Comment 8

[Ryan VanderMeulen [:RyanVM]]

Fixed by bug 1799460.