[wpt-sync] Sync PR 19383 - [LayoutNG] Fix replaced abspos with no intinsic size
Categories
(Core :: CSS Parsing and Computation, task, P4)
Tracking
()
Tracking | Status | |
---|---|---|
firefox71 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 19383 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/19383
Details from upstream follow.
Aleks Totic <atotic@chromium.org> wrote:
[LayoutNG] Fix replaced abspos with no intinsic size
One more abspos edge case:
Replaced element, with no intrinsic size, but with
intrinsic aspect ratio.The example problem here is an SVG with aspect ratio,
but no intinsic size. According to the spec, its
inline size should fill container, and block size
should be derived from inline size and aspect ratio.Existing code did not handle this case. If element
had no size, its size would be computed and get clamped
by minmax.A replaced element with no size gets a default minmax of
300x150, so elements would effectively get clamped to minmax.My fix is a bit of a hack. It uses absence of minmax as a signal
that size should not be clamped.The intrinsic size handling is surprisingly complex for replaced
elements. I wonder if there are any other parts of NG that
need a close look in how replaced with aspect ratio/no size
are handled.FF handles this correctly. Legacy does not.
Bug: 1003545
Change-Id: I1510611ff84a99a26bcade74cace2855e022f997
Reviewed-on: https://chromium-review.googlesource.com/1830326
WPT-Export-Revision: d7e2eba767bbc80d97b28781ae787631cb9c5d20
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=23b9c2335383df93b82932668ea512ab3a62447a
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=567d4aea0424e7cc45276d8d87996012029464d1
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=7396a60386cca170fe05049123b812d59a1fb792
Assignee | ||
Comment 4•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=0819506c343e6de55bb8fb7884308ab6b5033fc4
Assignee | ||
Comment 5•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=7a25938af7992caf9cd517cf94103f4246e28292
Assignee | ||
Comment 6•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=c8f4e9e4f931e5eca5d377d3c33d9bdf14caf9ab
Assignee | ||
Comment 7•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=55c81b1ce728791dcbd559acb670ffa961148e79
Assignee | ||
Comment 8•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=7680cbfa4e755888d063bc91629e3535097d8e98
Assignee | ||
Comment 9•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=d8bc7696c81ef323585ae2b3205127391df93ba1
Assignee | ||
Comment 10•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=1651bebc9fedc909634f1b476dab70e8993ae91d
Assignee | ||
Comment 11•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=635682c41c27f350480cb890034338214495bc19
Assignee | ||
Comment 12•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=3e1bafc25a52278421ca5d04fb7042a8199d4f3f
Assignee | ||
Comment 13•5 years ago
|
||
Ran 1 tests and 38 subtests OK : 1 PASS : 38
Comment 14•5 years ago
|
||
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/3e9b74acb717 [wpt PR 19383] - [LayoutNG] Fix replaced abspos with no intrinsic size, a=testonly
Comment 15•5 years ago
|
||
bugherder |
Description
•