Closed
Bug 1514658
Opened 5 years ago
Closed 5 years ago
Fix a few baseline alignment tests that makes bogus assumptions about line-height metrics
Categories
(Core :: Layout: Grid, defect, P4)
Core
Layout: Grid
Tracking
()
RESOLVED
FIXED
mozilla66
Tracking | Status | |
---|---|---|
firefox66 | --- | fixed |
People
(Reporter: MatsPalmgren_bugz, Assigned: MatsPalmgren_bugz)
References
Details
Attachments
(1 file)
5.42 KB,
patch
|
emilio
:
review+
|
Details | Diff | Splinter Review |
The following WPT tests are bogus because they depend on line-height metrics, which differs between UAs: /css/css-grid/alignment/grid-alignment-style-changes-005.html /css/css-grid/alignment/grid-alignment-style-changes-006.html /css/css-grid/alignment/grid-alignment-style-changes-007.html /css/css-grid/alignment/grid-alignment-style-changes-008.html
Assignee | ||
Comment 1•5 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=5aaebe61fe010e38920e7ac6844006d91c7bd105
Assignee | ||
Comment 2•5 years ago
|
||
Attachment #9031991 -
Flags: review?(emilio)
Comment 3•5 years ago
|
||
Comment on attachment 9031991 [details] [diff] [review] fix Review of attachment 9031991 [details] [diff] [review]: ----------------------------------------------------------------- Looks fair, thanks!
Attachment #9031991 -
Flags: review?(emilio) → review+
Pushed by mpalmgren@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/21c134f346bb Fix a few baseline alignment tests that makes bogus assumptions about line-height metrics. r=emilio
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/14578 for changes under testing/web-platform/tests
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Comment 7•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/21c134f346bb
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox66:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
Upstream PR merged
You need to log in
before you can comment on or make changes to this bug.
Description
•