Closed
Bug 1824256
Opened 3 years ago
Closed 3 years ago
[wpt-sync] Sync PR 39172 - Add CSSMathExpressionNumericLiteral canonical units eager simplification
Categories
(Core :: DOM: CSS Object Model, task, P4)
Core
DOM: CSS Object Model
Tracking
()
RESOLVED
FIXED
114 Branch
| Tracking | Status | |
|---|---|---|
| firefox114 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Attachments
(11 files)
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details | |
|
464.48 KB,
text/plain
|
Details |
Sync web-platform-tests PR 39172 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/39172
Details from upstream follow.
Traian Captan <tcaptan@chromium.org> wrote:
Add CSSMathExpressionNumericLiteral canonical units eager simplification
Spec def:
"If root is a dimension that is not expressed in its canonical unit, and
there is enough information available to convert it to the canonical
unit, do so, and return the value."
https://w3c.github.io/csswg-drafts/css-values/#calc-simplificationHowever, Numbers should not be eagerly simplified at this time since
that would result in converting Integers to Doubles (kNumber, canonical
unit for Numbers).R=xiaochengh
Bug: 1050968
Change-Id: I79adfbd516be640be51b86e1b33a2693c3b51ed9
Reviewed-on: https://chromium-review.googlesource.com/4367363
WPT-Export-Revision: 0ae699ff41143b1148e7bc7651191e999f13c694
|
Assignee | |
Updated•3 years ago
|
Component: web-platform-tests → DOM: CSS Object Model
Product: Testing → Core
|
|
Assignee | |
Comment 1•3 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95
|
|
Assignee | |
Comment 2•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 3•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 4•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 5•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 6•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 7•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 8•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 9•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 10•3 years ago
|
||
Test result changes from PR not available.
|
|
Assignee | |
Comment 11•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 12•3 years ago
|
||
Test result changes from PR not available.
|
|
Assignee | |
Comment 13•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 14•3 years ago
|
||
Test result changes from PR not available.
|
||
Comment 15•3 years ago
|
||
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e50c46aefa9e
[wpt PR 39172] - Add CSSMathExpressionNumericLiteral canonical units eager simplification, a=testonly
|
|
Assignee | |
Comment 16•3 years ago
|
||
# CI Results
Ran 9 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI
Total 240 tests and 32 subtests
## Status Summary
### Firefox
`ERROR`: 240
### Chrome
`OK` : 235
`PASS` : 10456
`FAIL` : 659
`ERROR`: 5
### Safari
`OK` : 235
`PASS` : 10481
`FAIL` : 634
`ERROR`: 5
## Links
[Gecko CI (Treeherder)](https://treeherder.mozilla.org/#/jobs?repo=try&revision=1205ff5c9b7a84d76f4a2bd5dff06310f7b66a95)
[GitHub PR Head](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_head)
[GitHub PR Base](https://wpt.fyi/results/?sha=c01046ea5038c7d849efe37a635764202ed15c2f&label=pr_base)
## Details
(See attachment for full changes)
|
|
Assignee | |
Comment 17•3 years ago
|
||
Test result changes from PR not available.
|
||
Comment 18•3 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox114:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•