Closed Bug 1867081 Opened 2 years ago Closed 1 year ago

[wpt-sync] Sync PR 43391 - Fix progress function calculation

Categories

(Core :: CSS Parsing and Computation, task, P4)

Product:
Core
Component:
CSS Parsing and Computation
Type:
task

Tracking

()

Status:
RESOLVED FIXED
Milestone:
122 Branch
Tracking Flags:
Tracking Status
firefox122 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 43391 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/43391
Details from upstream follow.

Daniil Sakhapov <sakhapov@chromium.org> wrote:

Fix progress function calculation

We can simplify the progress function when either all the operands
resolve to the same canonical unit or all the operands have the same type.
Currently progress(10em from 10em to 100em) fails as we try to compute the
canonical value in either case of simplification.

This CL changes this behavior to distinguish which simplification path
to use, so that in the mentioned progress() example we just take the
double values when all the operands are of the same type.

Bug: 1503730
Change-Id: I361477ba2a03fd6063861d1d1a82d4b77fa77f2f
Reviewed-on: https://chromium-review.googlesource.com/5067615
WPT-Export-Revision: 5533504606c3c91bfac93ccb056a15a5bb6d4cee

PR 43391 applied with additional changes from upstream: ef56b263783f156edf26f3f8c79753e53dfc07cd
Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core
Test result changes from PR not available.
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/7d69e94414fa [wpt PR 43391] - Fix progress function calculation, a=testonly https://hg.mozilla.org/integration/autoland/rev/5cbd413189d8 [wpt PR 43391] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/7d69e94414fa
https://hg.mozilla.org/mozilla-central/rev/5cbd413189d8

Status: NEW → RESOLVED
Closed: 1 year ago
status-firefox122: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 122 Branch
You need to log in before you can comment on or make changes to this bug.