Closed Bug 800099 Opened 7 years ago Closed 7 years ago

scroll-rounding reftest requires > 600x600 resolution

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla19

People

(Reporter: jmaher, Assigned: jmaher)

References

Details

Attachments

(1 file)

We are changing the maximum size of the reftests to be 600x600, which means text-overflow/scroll-rounding.html needs to be adjusted.
Assignee: nobody → jmaher
Status: NEW → ASSIGNED
Attachment #670087 - Flags: review?(matspal)
Comment on attachment 670087 [details] [diff] [review]
adjust scroll-rounding.html to be< 600x600 (1.0)

In general, I think removing parts of tests that doesn't fit the new size
is the wrong approach because the parts you remove might be the only test
we have for some odd edge case.

In this case though, I think testing 100.1 100.2 100.3 and 100.4px
is probably overkill - but I would like to keep the end points.
So, in addition to your changes, please change the .t3 rule to
100.4px and remove the .t4 rule (not used).

r=mats with those changes
Attachment #670087 - Flags: review?(matspal) → review+
https://hg.mozilla.org/mozilla-central/rev/22d51507ec02
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in before you can comment on or make changes to this bug.