Closed
Bug 800099
Opened 13 years ago
Closed 13 years ago
scroll-rounding reftest requires > 600x600 resolution
Categories
(Core :: CSS Parsing and Computation, defect)
Core
CSS Parsing and Computation
Tracking
()
RESOLVED
FIXED
mozilla19
People
(Reporter: jmaher, Assigned: jmaher)
References
Details
Attachments
(1 file)
|
2.41 KB,
patch
|
MatsPalmgren_bugz
:
review+
|
Details | Diff | Splinter Review |
We are changing the maximum size of the reftests to be 600x600, which means text-overflow/scroll-rounding.html needs to be adjusted.
|
Assignee | |
Comment 1•13 years ago
|
||
|
||
Comment 2•13 years ago
|
||
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+
|
|
Assignee | |
Comment 3•13 years ago
|
||
|
||
Comment 4•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla19
You need to log in
before you can comment on or make changes to this bug.
Description
•