Skip to content

Commit

Permalink
Bug 1687849 [wpt PR 27265] - wpt computing-row-measure-0.html fix, a=…
Browse files Browse the repository at this point in the history
…testonly

Automatic update from web-platform-tests
wpt computing-row-measure-0.html fix

2 tests in this test suite seem inconsistent:

test#2 asserts that

tbody.height=10px > tr.height=1px > td.height=1px
implies td.offsetHeight = 1px

test#4 asserts that

tbody.height=10px > tr > td.height=1px
implies td.offsetHeight = 10px

Edge 17 is the only browser that agrees with #2 and #4
FF agrees with #2, but not #4
Chrome agrees with #4, but not #2
Safari agrees with #4, but not #2

To me, #2 and #4 seem to be in conflict.
Either tbody height propagates to rows, or it does not.

The problem is that #2 is overconstrained.

My suggestion is that tbody height always propagates to tr.

Bug: 958381
Change-Id: I28bfd108c67968d31d0372b536c316c997d2d958
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2586097
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#845515}

--

wpt-commits: 133ca044f481c49a8efd3f14f34f4b6a1b316f58
wpt-pr: 27265
  • Loading branch information
Aleks Totic authored and moz-wptsync-bot committed Jan 22, 2021
1 parent b06d5e2 commit 2fabf72
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ <h4>"Computing row measures"</h4>
[
"Checking intermediate min-content height for span 1 (2)",
document.getElementById('two').offsetHeight,
1
10
],
[
"Checking intermediate min-content height for span 1 (3)",
Expand Down

0 comments on commit 2fabf72

Please sign in to comment.