Closed Bug 1232979 Opened 7 years ago Closed 7 years ago

abs-pos/scrollframe-2.html fails on new linux64 docker container

Categories

(Taskcluster :: Services, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla46

People

(Reporter: jmaher, Assigned: jmaher)

References

Details

Attachments

(1 file)

this is the error we get in the log file:
REFTEST TEST-UNEXPECTED-FAIL | file:///home/worker/workspace/build/tests/reftest/tests/layout/reftests/abs-pos/scrollframe-2.html | image comparison (==), max difference: 1, number of differing pixels: 1

you can see the difference here:
http://hg.mozilla.org/mozilla-central/raw-file/tip/layout/tools/reftest/reftest-analyzer.xhtml#logurl=https://queue.taskcluster.net/v1/task/Og4ayYWjQOC4dwpC3js6Rw/runs/0/artifacts/public/logs/live_backing.log&only_show_unexpected=1

I suspect this is a small issue similar to bug 1232316.  For reference, the try run is here:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=0a238c213d3a (we care about R1 in this case)

here is a link to the test file:
https://dxr.mozilla.org/mozilla-central/source/layout/reftests/abs-pos/scrollframe-2.html

and the reference:
https://dxr.mozilla.org/mozilla-central/source/layout/reftests/abs-pos/scrollframe-2-ref.html

this already has a lot of fuzzy conditions around it:
https://dxr.mozilla.org/mozilla-central/source/layout/reftests/abs-pos/reftest.list#52

if we were to solve this with a fuzzy-if condition we could add:
fuzzy-if(gtkWidget,1,1)
:ehsan, do you have ideas on the proper fix here?  as you can see we have done some fuzzy-if in the past.  Other techniques are to increase the font size.  maybe in this case we have incorrect font packages.
Flags: needinfo?(ehsan)
I guess you can adjust the fuzzy pixel count or some such.
Flags: needinfo?(ehsan)
going the simple route of adding a fuzzy-if for linux.  Tested that this doesn't affect our existing automation and fixes our new automation.
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8699406 - Flags: review?(ehsan)
Comment on attachment 8699406 [details] [diff] [review]
add fuzzy-if for failing test case

Review of attachment 8699406 [details] [diff] [review]:
-----------------------------------------------------------------

rs=me
Attachment #8699406 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/f569c7e03c23
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
Moving closed bugs across to new Bugzilla product "TaskCluster".
Component: TaskCluster → Integration
Product: Testing → Taskcluster
Component: Integration → Services
You need to log in before you can comment on or make changes to this bug.