Closed Bug 1839928 Opened 3 years ago Closed 3 years ago

[wpt-sync] Sync PR 40703 - cc: Remove filter condition from the clipping code path.

Categories

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

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

Tracking

()

Status:
RESOLVED FIXED
Milestone:
117 Branch
Tracking Flags:
Tracking Status
firefox117 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

Details

(Whiteboard: [wptsync downstream])

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

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

Vladimir Levin <vmpstr@chromium.org> wrote:

cc: Remove filter condition from the clipping code path.

It doesn't seem to be correct to clip only in the presence
of filters (and is incorrect in the newly added case). Only
one other case failed with subpixel differences, but I think
that's acceptable.

Bug: 1444622
Change-Id: Ie811a3a8acc1dbce25d87b4fc84bb32ff0df1adc

Reviewed-on: https://chromium-review.googlesource.com/4633901
WPT-Export-Revision: 563192f6af3ac3ba084f7938b638a6f4c2c7546e

Component: web-platform-tests → CSS Parsing and Computation
Product: Testing → Core

CI Results

Ran 0 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 1 tests

Status Summary

Firefox

TIMEOUT: 1

Chrome

FAIL : 1

Safari

TIMEOUT: 1

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8082fcd229c0 [wpt PR 40703] - cc: Remove filter condition from the clipping code path., a=testonly https://hg.mozilla.org/integration/autoland/rev/3b16b8a8565f [wpt PR 40703] - Update wpt metadata, a=testonly

https://hg.mozilla.org/mozilla-central/rev/8082fcd229c0
https://hg.mozilla.org/mozilla-central/rev/3b16b8a8565f

Status: NEW → RESOLVED
Closed: 3 years ago
status-firefox117: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
You need to log in before you can comment on or make changes to this bug.