unexpected variations in width of <input type=number> when running reftests

RESOLVED FIXED in Firefox 38

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jfkthame, Assigned: jfkthame)

Tracking

(Blocks 1 bug)

unspecified
mozilla39
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox38 fixed, firefox39 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

4 years ago
The horizontal writing-mode examples of number controls in reftests/writing-mode/ua-style-sheet-input-number-1.html display as expected when the testcase and reference are loaded locally. However, when they're run by the reftest framework, the number fields in the testcase vary significantly in width (in varying ways across different platforms). See the test failures in https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=e37d57033b13.

I'm proposing to mark this test as failing for now (note that its vertical writing-mode portion is commented out anyway, due to bug 1123299), so that we can proceed to enable the rest of the writing-mode test directory. Offhand, I don't see an obvious reason why this test is misbehaving, but we should investigate and resolve this, perhaps along with 1123299.
Assignee

Comment 1

4 years ago
The number-control.css file belongs logically as a part of forms.css, and should also be loaded as a UA sheet with the same access to properties. Without this, we get an unexpected difference in behavior (in horizontal writing mode) depending whether the vertical-text pref is enabled, as that determines whether the inline-size property is recognized or not.
Attachment #8568004 - Flags: review?(cam)
Assignee

Updated

4 years ago
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Assignee

Updated

4 years ago
Blocks: 974430
Assignee

Comment 2

4 years ago
And this should allow us to remove the "fails" annotation from reftests/writing-mode/ua-style-sheet-input-number-1.html, which is failing because the inline-size property is discarded. I've pushed a try job to check; this should report "unexpected passes" on that test: https://treeherder.mozilla.org/#/jobs?repo=try&revision=9aced5cf3e6b.
Attachment #8568004 - Flags: review?(cam) → review+
https://hg.mozilla.org/mozilla-central/rev/2cf3beee5149
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Assignee

Comment 5

4 years ago
Comment on attachment 8568004 [details] [diff] [review]
The number-control.css stylesheet should be treated as a UA sheet when loaded, so that the inline-size property is recognized

Approval Request Comment
[Feature/regressing bug #]: bug 1119475
[User impact if declined]: UA stylesheet fails to set the intended default width for <input type="number"> elements.
[Describe test coverage new/current, TreeHerder]: Reftest is in the tree, though currently disabled; will be enabled in bug 1134598 once the leak that forced a backout there is resolved. (Local testing and tryserver runs confirm the reftest will pass when it gets enabled.)
[Risks and why]: minimal
[String/UUID change made/needed]: n/a
Attachment #8568004 - Flags: approval-mozilla-aurora?
Comment on attachment 8568004 [details] [diff] [review]
The number-control.css stylesheet should be treated as a UA sheet when loaded, so that the inline-size property is recognized

Approving for uplift to aurora since it sounds low risk. It would be good to get the test coverage uplifted as well before this goes to beta.
Attachment #8568004 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.