Closed Bug 1232981 Opened 7 years ago Closed 7 years ago

bidi/83958-1*.html reftests fail 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)

in pushing to try where we run taskcluster jobs in a docker container, we have some reftest failures:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=0a238c213d3a&filter-searchStr=r1


REFTEST TEST-UNEXPECTED-FAIL | file:///home/worker/workspace/build/tests/reftest/tests/layout/reftests/bidi/83958-1a.html | image comparison (==), max difference: 1, number of differing pixels: 12
REFTEST TEST-UNEXPECTED-FAIL | file:///home/worker/workspace/build/tests/reftest/tests/layout/reftests/bidi/83958-1b.html | image comparison (==), max difference: 1, number of differing pixels: 18
REFTEST TEST-UNEXPECTED-FAIL | file:///home/worker/workspace/build/tests/reftest/tests/layout/reftests/bidi/83958-1c.html | image comparison (==), max difference: 1, number of differing pixels: 18 

reftest analyzer shows a slight difference (not very easy to see to the eye):
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


If we wanted to adjust the manifest for fuzzing, we would add:
fuzzy-if(gtkWidget,1,12) and fuzzy-if(gtkWidget,1,18)

right now these tests have no existing fuzzy-if statements in the manifest:
https://dxr.mozilla.org/mozilla-central/source/layout/reftests/bidi/reftest.list#50

an example of the 1a test-
test: https://dxr.mozilla.org/mozilla-central/source/layout/reftests/bidi/83958-1a.html
ref: https://dxr.mozilla.org/mozilla-central/source/layout/reftests/bidi/83958-1-ref.html
Should we NI for this bug? or not yet?
What's "NI"?
It might make sense to just add some letter-spacing (maybe 1px or 2px) to the tests, assuming that that works...
NI: needinfo

Hi David! I hadn't checked the CC list. Thanks for your input!
I couldn't get letter-spacing: 1|2px working for either the test and/or the reference.  I did try adding:
body { font-size: 300%; }

this added to both the test and the reference proves success.  This is a suggestion that was said in bug 1232980 and it worked there.
:dbaron, if you don't like this approach of font-size, then please cancel the review.  I can easily reproduce/hack on this locally if you have other suggestions or preferences.  Otherwise it will be nice to get this in!
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #8701063 - Flags: review?(dbaron)
Did you check that the test still fits within the window (i.e., the screenshotted area)?
Flags: needinfo?(jmaher)
ok, figured out how to get this by dumping the dodataUrl() for successful tests and then putting that into a img src:
http://people.mozilla.org/~jmaher/bidi.html

Please let me know if there is a better way to do this and if this answers your question.  I believe this is showing all the data we expect.
Flags: needinfo?(jmaher)
Comment on attachment 8701063 [details] [diff] [review]
add font-size:300 to the body to reduce fuzzy painting

r=dbaron if you also add:

p { margin: 0 }

so that it will still fit if/when we switch to 600x600


(The easy way to look at the images is to change the test to a != and look at the failure in reftest-analyzer.xhtml.)
Attachment #8701063 - Flags: review?(dbaron) → review+
https://hg.mozilla.org/mozilla-central/rev/6b5c53463b1e
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.