Fix box-sizing/intrinsic-1* reftests to not use "box-sizing: padding-box"

RESOLVED FIXED in Firefox 50

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: dholbert, Assigned: dholbert)

Tracking

Trunk
mozilla50
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox50 fixed)

Details

(As it happens, these tests don't actually have any padding, so "box-sizing:padding-box" works out to be the same as "content-box" [the default value of the property], in practice. So bug 1166728's unsupporting doesn't actually affect the rendering of these tests.)
Oops, I forgot to do this before landing bug 1166728. I'll land it as a followup instead.

We can just drop the "box-sizing" declarations from the tests, since padding-box is equivalent to content-box in these cases.

(I wasn't sure if we should make that tweak, or if that would just make these tests identical to another existing testcase alongside them in the a,b,c,...o listing of tests here [in which case we'd just want to remove them].  I've now verified that it does *not* make them identical to another such test -- all of the sibling-tests use "box-sizing:border-box" except for these 3. So they're still unique, post-tweak.)
Pushed by dholbert@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3c5025f98e56
Remove unneeded "box-sizing: padding-box" CSS from 3 reftests, now that it's become invalid CSS. (test only, no review.)
https://hg.mozilla.org/mozilla-central/rev/3c5025f98e56
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.