Closed Bug 1618855 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 22019 - shape-outside test slightly incorrect

Categories

(Core :: Layout: Floats, task, P4)

task

Tracking

()

RESOLVED FIXED
mozilla75
Tracking Status
firefox75 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

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

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

Mike <mike@qtunes.org> wrote:

shape-outside test slightly incorrect

This test is trying to position Ahem text (100px wide) next to a float which is cropped to an effective width of 110px (the 100px width of the shape from the image, expanded by a 10px margin in all directions - but we only care about the right margin here). So the left edge of the text relative to its container is 110px.

But, the container it was supposed to go into was set to 200px wide. It needs to be at least 210px wide to accommodate it, otherwise the text will wrap to the next line and the test fails. I've set it to 211px to allow for 1px latitude retrieving the mask from the image (note that the red box is testing the exact position - so long as the width of the container is wide enough to fit the image, it's exact width shouldn't impact on the test)

Component: web-platform-tests → Layout: Floats
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

FAIL: 1

Chrome

FAIL: 1

Safari

FAIL: 1

Links

GitHub PR Head
GitHub PR Base

Details

New Tests That Don't Pass

/css/css-shapes/shape-outside/shape-image/shape-image-024.html: FAIL (Chrome: FAIL, Safari: FAIL)

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6b93955cb509 [wpt PR 22019] - shape-outside test slightly incorrect, a=testonly
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
You need to log in before you can comment on or make changes to this bug.