Closed Bug 1232981 Opened 8 years ago Closed 8 years ago

bidi/83958-1*.html reftests fail on new linux64 docker container


(Taskcluster :: Services, defect)

Not set


(Not tracked)



(Reporter: jmaher, Assigned: jmaher)




(1 file)

in pushing to try where we run taskcluster jobs in a docker container, we have some reftest failures:

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):

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:

an example of the 1a test-
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
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:

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+
Closed: 8 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.