Closed Bug 755025 Opened 12 years ago Closed 12 years ago

Intermittent REFTEST TEST-UNEXPECTED-FAIL | xulscroll.html and/or textbox-setsize.xul | image comparison (==)

Categories

(Core :: Layout, defect)

15 Branch
x86
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: mbrubeck, Assigned: MatsPalmgren_bugz)

References

(Depends on 1 open bug)

Details

(Keywords: intermittent-failure)

Attachments

(1 file)

Looks like a focus or mouse hover issue.  In scrollbox.html one of the scrollbars in image 1 is highlighted, while in image 2 none of them are.  In textbox-setsize.xul, the textbox in image 1 has its border highlighted in blue, while image 2 has a grey border.

https://tbpl.mozilla.org/php/getParsedLog.php?id=11732043&tree=Mozilla-Inbound
Rev3 WINNT 6.1 mozilla-inbound opt test reftest-no-accel on 2012-05-14 11:55:54 PDT for push b001da365ab7
slave: talos-r3-w7-023

REFTEST INFO | Loading a blank page
REFTEST TEST-START | http://localhost:4444/1337022016745/188/text-overflow/xulscroll.html | 3522 / 7608 (46%)
REFTEST TEST-START | http://localhost:4444/1337022016745/188/text-overflow/xulscroll-ref.html | 3522 / 7608 (46%)
REFTEST TEST-UNEXPECTED-FAIL | file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/text-overflow/xulscroll.html | image comparison (==)
REFTEST number of differing pixels: 2548 max difference: 106
REFTEST INFO | Loading a blank page

REFTEST INFO | Loading a blank page
REFTEST TEST-START | file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/forms/textbox-setsize.xul | 4222 / 7608 (55%)
REFTEST TEST-START | file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/forms/textbox-setsize-ref.xul | 4222 / 7608 (55%)
REFTEST TEST-UNEXPECTED-FAIL | file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/forms/textbox-setsize.xul | image comparison (==)
REFTEST number of differing pixels: 2000 max difference: 84
REFTEST INFO | Saved log: START file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/forms/textbox-setsize.xul
REFTEST INFO | Saved log: [CONTENT] OnDocumentLoad triggering AfterOnLoadScripts
REFTEST INFO | Saved log: Initializing canvas snapshot
REFTEST INFO | Saved log: DoDrawWindow 0,0,800,1000
REFTEST INFO | Saved log: [CONTENT] RecordResult fired
REFTEST INFO | Saved log: RecordResult fired
REFTEST INFO | Saved log: START file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/forms/textbox-setsize-ref.xul
REFTEST INFO | Saved log: [CONTENT] OnDocumentLoad triggering AfterOnLoadScripts
REFTEST INFO | Saved log: Initializing canvas snapshot
REFTEST INFO | Saved log: DoDrawWindow 0,0,800,1000
REFTEST INFO | Saved log: [CONTENT] RecordResult fired
REFTEST INFO | Saved log: RecordResult fired
REFTEST INFO | Loading a blank page

https://tbpl.mozilla.org/php/getParsedLog.php?id=11734449&tree=Firefox
Rev3 WINNT 6.1 mozilla-central opt test reftest on 2012-05-14 13:11:13 PDT for push 498d2784a240
slave: talos-r3-w7-031

REFTEST INFO | Loading a blank page
REFTEST TEST-START | http://localhost:4444/1337026372491/188/text-overflow/xulscroll.html | 3522 / 7608 (46%)
REFTEST TEST-START | http://localhost:4444/1337026372491/188/text-overflow/xulscroll-ref.html | 3522 / 7608 (46%)
REFTEST TEST-UNEXPECTED-FAIL | file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/text-overflow/xulscroll.html | image comparison (==)
REFTEST number of differing pixels: 2458 max difference: 106
REFTEST INFO | Loading a blank page
AFAIK, the reftest framework doesn't guarantee that the synthetic mouse event
that triggers :hover on document load has been run, so I suppose there's a race
between that and the snapshot being taken.  We should probably fix this in the
framework somehow.
If the tests aren't supposed to be testing hover, the best thing to do is to set styles on the tests so that they don't respond to hover. That way they won't fail when someone is running tests locally and accidentally moves the mouse over the window.
Hmm, is there a way to get a non-themed scrollbar for a <div>?
I tried "div { -moz-appearance: none; }" but it didn't have any effect.

I still regard this as a bug in the reftest framework - it should run
the test and the reference in exactly the same way.
Attached patch wallpaperSplinter Review
https://tbpl.mozilla.org/?usebuildbot=1&tree=Try&rev=2002a074e86e
(the Android R3 oranges are unrelated)
Attachment #623997 - Flags: review?(roc)
https://hg.mozilla.org/integration/mozilla-inbound/rev/7cf218423f9c

Filed follow-up bug 755955.
Target Milestone: --- → mozilla15
Depends on: 755955
https://hg.mozilla.org/mozilla-central/rev/7cf218423f9c
Assignee: nobody → matspal
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Whiteboard: [orange]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: