752864 - Intermittent t1204-reset-00-c-o-test.html | image comparison (==)

Status: RESOLVED FIXED

(Core :: CSS Parsing and Computation, defect)

Description

[Ed Morley [:emorley]]

Attachment: reftest log

Rev3 Fedora 12x64 mozilla-inbound pgo test reftest on 2012-05-07 19:11:01 PDT for push 7d96ff6113c7

slave: talos-r3-fed64-024

https://tbpl.mozilla.org/php/getParsedLog.php?id=11552429&tree=Mozilla-Inbound

{
REFTEST TEST-START | file:///home/cltbld/talos-slave/test/build/reftest/tests/layout/reftests/counters/t1204-reset-00-c-o-test.html | 3797 / 7597 (49%)
REFTEST TEST-START | file:///home/cltbld/talos-slave/test/build/reftest/tests/layout/reftests/counters/t1204-reset-00-c-o-reference.html | 3797 / 7597 (49%)
REFTEST TEST-UNEXPECTED-FAIL | file:///home/cltbld/talos-slave/test/build/reftest/tests/layout/reftests/counters/t1204-reset-00-c-o-test.html | image comparison (==)
...
REFTEST number of differing pixels: 176 max difference: 255
REFTEST INFO | Saved log: START file:///home/cltbld/talos-slave/test/build/reftest/tests/layout/reftests/counters/t1204-reset-00-c-o-test.html
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:///home/cltbld/talos-slave/test/build/reftest/tests/layout/reftests/counters/t1204-reset-00-c-o-reference.html
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
}

Comment 1

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

For the record, this test is among the oldest reftests.  If this happens again, we should look at the regression range.

Comment 2

[Legacy TBPL/Treeherder Robot]

philor
https://tbpl.mozilla.org/php/getParsedLog.php?id=16164767&tree=Mozilla-Inbound
Rev3 WINNT 5.1 mozilla-inbound opt test reftest on 2012-10-16 11:07:32
slave: talos-r3-xp-046

REFTEST TEST-UNEXPECTED-FAIL | file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/counters/t1204-reset-00-c-o-test.html | image comparison (==), max difference: 255, number of differing pixels: 101

Comment 3

[Legacy TBPL/Treeherder Robot]

philor
https://tbpl.mozilla.org/php/getParsedLog.php?id=16500996&tree=Fx-Team
Rev3 WINNT 5.1 fx-team debug test reftest on 2012-10-26 13:07:51
slave: talos-r3-xp-039

REFTEST TEST-UNEXPECTED-FAIL | file:///c:/talos-slave/test/build/reftest/tests/layout/reftests/counters/t1204-reset-00-c-o-test.html | image comparison (==), max difference: 255, number of differing pixels: 101

Comment 4

[Nathan Froyd [:froydnj]]

Attachment: patch

As discussed on IRC, this test should be using reftest-wait

Comment 5

[Daniel Holbert [:dholbert]]

Comment on attachment 675906 [details] [diff] [review]
patch

># HG changeset patch
># User Nathan Froyd <[e-mail]>
>
>fix layout/reftests/counters/t1204-reset-00-c-o-test.html intermittent orange by using MozReftestInvalidate

As in bug 778695, commit message should mention reftest-wait.

(Also, if you like, you don't need to include the full path to the test in the commit message -- just the filename is probably fine.  It's almost certainly unique, and even if it weren't, the actual file-path info is included in the cset itself, so people can find it there if they care.  Doesn't really matter though; feel free to keep the path if you like.)

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/7e1b636e80e3

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/95315ccdbb7d
https://hg.mozilla.org/releases/mozilla-beta/rev/5dcf75a41554