Closed
Bug 1516743
Opened 5 years ago
Closed 5 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•5 years ago
|
Component: web-platform-tests → SVG
Product: Testing → Core
Assignee | ||
Comment 1•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=15726d830ec6340e8e16a64da82f27c3d8048b66
Assignee | ||
Comment 2•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=e21aa34c4764eadd78643e987bf702538ba169ed
Assignee | ||
Comment 3•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=a6a5e87af14aab0fc569a3f291d783621c85d050
Assignee | ||
Comment 4•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=ed388d3793c7e6d4ffb3d8ff331bc6dd2219e0f7
Assignee | ||
Comment 5•5 years ago
|
||
Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=904a63720f7f03980e6444f48e80f3bf0ccee955
Assignee | ||
Comment 6•5 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=49fd5fc3fb3ca1803869165cb36b18fc4b55c72a
Assignee | ||
Comment 7•5 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•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 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
•