Skip to content

Commit

Permalink
Bug 1560872 [wpt PR 17381] - [LayoutNG] Skip "replaced % resolution" …
Browse files Browse the repository at this point in the history
…quirk on ortho-roots., a=testonly

Automatic update from web-platform-tests
[LayoutNG] Skip "replaced % resolution" quirk on ortho-roots.

This is about an undocumented quirk. I raised
whatwg/quirks#46 for it.

LayoutNG currently cannot handle this situation, because we don't store
the inline-size to use in replaced percentage resolution (only the
block-size). However, I find it hard to believe that there'd be any
specific expectations to how this is supposed to work if a table cell
has a definite block-size, and a descendant establishes an orthogonal
writing-mode root. So just ignore it and use regular percentage size
resolution in these cases.

Bug: 967069
Change-Id: Iac61abe80f4f949017cc161855b2223ec4503812
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1664075
Reviewed-by: Ian Kilpatrick <ikilpatrickchromium.org>
Reviewed-by: Christian Biesinger <cbiesingerchromium.org>
Commit-Queue: Morten Stenshorne <mstenshochromium.org>
Cr-Commit-Position: refs/heads/master{#670126}

--

wpt-commits: 361912dbe4ef692ea256b6f5bc0d74c0668ccdf5
wpt-pr: 17381

UltraBlame original commit: 404f92fcf1dd5294fae238edc14f184e1cdcd6a0
  • Loading branch information
marco-c committed Oct 4, 2019
1 parent c75f37a commit e873489
Showing 1 changed file with 14 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@

<link rel="author" title="Morten Stenshorne" href="mailto:[email protected]">
<link rel="help" href="https://github.com/whatwg/quirks/issues/46">
<link rel="help" href="https://bugs.chromium.org/p/chromium/issues/detail?id=967069">
<div style="display:table-cell; height:10000px;">
<div style="writing-mode:vertical-rl;">
<iframe style="height:100%;"></iframe>
</div>
</div>
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<script>
test(()=> { }, "No crash");
</script>

0 comments on commit e873489

Please sign in to comment.