Closed Bug 849771 Opened 7 years ago Closed 7 years ago

layout/reftests/bugs ref test failures on Win8

Categories

(Core :: Layout, defect)

Product:
Core
Component:
Layout
Platform:
x86_64
Windows 8
Type:
defect
Priority:
Not set

Tracking

()

Status:
RESOLVED FIXED

People

(Reporter: jimm, Assigned: jimm)

References

Details

Attachments

(1 file)

partial patch
2.35 KB, patch
roc
: review+
Details | Diff | Splinter Review 
this marked as random on win7:

12:59:39 INFO - REFTEST TEST-UNEXPECTED-FAIL | file:///C:/slave/test/build/tests/reftest/tests/layout/reftests/bugs/379316-2.html | image comparison (==), max difference: 1, number of differing pixels: 150

test failure:

13:02:14 INFO - REFTEST TEST-UNEXPECTED-FAIL | file:///C:/slave/test/build/tests/reftest/tests/layout/reftests/bugs/759036-1.html | image comparison (==), max difference: 1, number of differing pixels: 77608

this test needs the fuzzy value upped a bit, it's currently 800:

13:02:16 INFO - REFTEST TEST-UNEXPECTED-FAIL | file:///C:/slave/test/build/tests/reftest/tests/layout/reftests/bugs/797797-1.html | image comparison (==), max difference: 1, number of differing pixels: 830
> test failure:
> 
> 13:02:14 INFO - REFTEST TEST-UNEXPECTED-FAIL |
> file:///C:/slave/test/build/tests/reftest/tests/layout/reftests/bugs/759036-
> 1.html | image comparison (==), max difference: 1, number of differing
> pixels: 77608

Locally I get a pixel difference of 23. I also get a failure on the follow up test 759036-2.html with a 5887 pixel difference.
Attached patch partial patch — — Splinter Review 
This fixes two of the three by changing the fuzzy values. On the third, I'd like to wait until bug 848936 is fixed.
Assignee: nobody → jmathies
Attachment #723409 - Flags: review?(roc)
Whiteboard: [leave-open]
splitting 759036-1.html into a new own bug.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [leave-open]
You need to log in before you can comment on or make changes to this bug.