Closed
Bug 1516743
Opened 6 years ago
Closed 6 years ago
[wpt-sync] Sync PR 14687 - [LayoutNG] Fix SVG replaced element sizes
Categories
(Core :: SVG, enhancement, P4)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla67
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: mozilla.org, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 14687 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/14687
Details from upstream follow.
Aleks Totic <atotic@chromium.org> wrote:
> [LayoutNG] Fix SVG replaced element sizes
>
> The old code made an incorrect assumption that if Legacy's
> IntrinsicSizingInfo did not compute width/height, the fallback
> should be LayoutBox::IntrinsicSize.
>
> I thought I was being clever, and was trying to keep sizing computation
> logic out of LayoutBox. Unfortunately, not enough information
> is exposed by LayoutBox to compute ReplacedSize from raw
> IntrinsicSize, and IntrinsicSizingInfo must be used.
>
> The correct fix is to use ConstraintSpace::AvailableSize if
> IntrinsicSizingInfo is unavailable.
>
> Bug: 916387
> Change-Id: Ib58bbd4775aa9fe1129bc5613d1652123b12670e
> Reviewed-on: https://chromium-review.googlesource.com/1392367
> WPT-Export-Revision: 76f7b398449bb0af974083e13f7658474a414130
Assignee | ||
Updated•6 years ago
|
Component: web-platform-tests → SVG
Product: Testing → Core
Assignee | ||
Comment 1•6 years ago
|
||
Assignee | ||
Comment 2•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=e21aa34c4764eadd78643e987bf702538ba169ed
Assignee | ||
Comment 3•6 years ago
|
||
Assignee | ||
Comment 4•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=ed388d3793c7e6d4ffb3d8ff331bc6dd2219e0f7
Assignee | ||
Comment 5•6 years ago
|
||
Assignee | ||
Comment 6•6 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=49fd5fc3fb3ca1803869165cb36b18fc4b55c72a
Assignee | ||
Comment 7•6 years ago
|
||
Ran 1 tests
PASS : 1
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9ff6bc6b7d90
[wpt PR 14687] - [LayoutNG] Fix SVG replaced element sizes, a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/939a9502b7ab
[wpt PR 14687] - [LayoutNG] Fix SVG replaced element sizes, a=testonly
Comment 10•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•