Closed Bug 587317 Opened 10 years ago Closed 10 years ago
[d2d] TEST-UNEXPECTED-FAIL | mochitest/general/test
_offsets .html | scrollbox scroll Height - got 127, expected 126
No description provided.
Rounding this value instead of flooring it seems like better behavior and works on both GDI and D2D.
Assignee: nobody → bas.schouten
Status: NEW → ASSIGNED
Correct patch this time.
Attachment #466077 - Attachment is obsolete: true
Comment on attachment 466078 [details] [diff] [review] Rounding instead of floor scrollHeight Roc gave his review for this on irc. You should file a follow-up bug to make this test saner by rounding correctly everywhere.
Attachment #466078 - Flags: review+
We cannot do this since it causes OS X mochitests to fail. Linux is fine though. I tried the rounding the whole thing at the end in checkCoord but that made GDI and D2D both fail. We should probably think hard about what we're supposed to be doing here.
This fixes it; passes on D2D, Mac and with GDI, and I think the code is logical too.
Comment on attachment 466192 [details] [diff] [review] fix Looks like this makes everything stricter (and also slightly different) except checkCoord (is that loosening necessary?), which is good. And if everything passes now, better.
Attachment #466192 - Flags: review?(dbaron) → review+
The checkCoord thing is safe (and needed) because scroll*, offset* and client* properties are never going to return non-integers, so if we've computed non-integer expected values we need to round those expected values the same way we round in our implementation.
10 years ago
Whiteboard: [needs landing]
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.