Closed Bug 1232979 Opened 10 years ago Closed 10 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+
Status: ASSIGNED → RESOLVED
Closed: 10 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.

Attachment

General

Created:
Updated:
Size: