Closed Bug 1291334 Opened 5 years ago Closed 5 years ago

Fix a few issues in w3c-submitted flexbox reftests

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: dholbert, Assigned: dholbert)

Details

Attachments

(2 files)

Filing this bug to fix some minor issues in our w3c-submitted flexbox reftests, reported by biesi over email.

(1)

https://github.com/w3c/csswg-test/blob/master/vendor-imports/mozilla/mozilla-central-reftests/flexbox/flexbox-basic-textarea-horiz-001.xhtml

https://github.com/w3c/csswg-test/blob/master/vendor-imports/mozilla/mozilla-central-reftests/flexbox/flexbox-basic-textarea-horiz-001-ref.xhtml

The question is about the second flexbox in this test. In the reference, that div has a height: 24px specified. But in the test itself, the flexbox has no explicit height and the textarea has a height: 20px plus a 1px border, adding to 22px. 

Apparently Firefox has a 1px block-start and block-end margin for textareas:
https://dxr.mozilla.org/mozilla-central/source/layout/style/res/forms.css#125

Would you be open to modifying that test to not rely on that? Either specify margin: 1px 0; or margin: 0;.

(2)
https://github.com/w3c/csswg-test/blob/master/vendor-imports/mozilla/mozilla-central-reftests/flexbox/flexbox-whitespace-handling-001a.xhtml
and
https://github.com/w3c/csswg-test/blob/master/vendor-imports/mozilla/mozilla-central-reftests/flexbox/flexbox-whitespace-handling-001b.xhtml

reference solidblue.png, even though it actually exists as support/solidblue.png. I
Patches up -- I triggered a try run:
 https://treeherder.mozilla.org/#/jobs?repo=try&revision=a993488772b6
...just in case the textarea margin change trips over some weird platform-specific widget rendering difference.  I'll land if the Try run goes well.
Pushed by dholbert@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3534790e9a67
part 1: Explicitly set margin on textareas within reftest flexbox-basic-textarea-horiz-001.xhtml, to remove assumptions about default styling. (no review)
https://hg.mozilla.org/integration/mozilla-inbound/rev/607ba3f1f6a1
part 2: Fix path to solidblue.png image in flexbox-whitespace-handling-001* reftest files. (no review)
https://hg.mozilla.org/mozilla-central/rev/3534790e9a67
https://hg.mozilla.org/mozilla-central/rev/607ba3f1f6a1
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
Normally I have a script that emails me each day if porting is needed.  But I can do it anytime, really...
You need to log in before you can comment on or make changes to this bug.