Skip to content

GitLab

T
tangled
Commit b23504c1 authored by Aleks Totic's avatar Aleks Totic Committed by Commit Bot
Browse files
Options

[LayoutNG] Dynamic height changes to abspos table 

Bug: 918904
Change-Id: I95483729d4f4839d18731ed73ae979a5917cf835
Reviewed-on: https://chromium-review.googlesource.com/c/1407119
Commit-Queue: Aleks Totic <atotic@chromium.org>
Reviewed-by: default avatarMorten Stenshorne <mstensho@chromium.org>
Cr-Commit-Position: refs/heads/master@{#622092}
parent dd614e74
Hide whitespace changes
Inline Side-by-side
Showing with 40 additions and 0 deletions
third_party/blink/renderer/core/layout/ng/ng_block_node.cc
View file @ b23504c1
......@@ -890,10 +890,14 @@ scoped_refptr<NGLayoutResult> NGBlockNode::RunOldLayout(
if (constraint_space.IsFixedSizeInline()) {
box_->SetOverrideLogicalWidth(
constraint_space.AvailableSize().inline_size);
} else {
box_->ClearOverrideLogicalWidth();
}
if (constraint_space.IsFixedSizeBlock()) {
box_->SetOverrideLogicalHeight(
constraint_space.AvailableSize().block_size);
} else {
box_->ClearOverrideLogicalHeight();
}
box_->ComputeAndSetBlockDirectionMargins(box_->ContainingBlock());
......
third_party/blink/web_tests/external/wpt/css/css-position/position-absolute-abspos-table-dynamic.html 0 → 100644
View file @ b23504c1
<!DOCTYPE html>
<title>CSS Position: dynamic abspos table height changes</title>
<link rel="author" title="mailto:atotic@google.com">
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<link rel="help" href="https://www.w3.org/TR/css-position-3/#abs-non-replaced-height">
<meta name="assert" content="Height changes for abspos table.">
<style>
#container {
width: 400px;
height: 300px;
background: green;
position: relative;
}
#target {
display: table;
background: red;
position: absolute;
top: 0;
width: 50%;
height: 100px;
}
</style>
<!-- There should be no red on this page -->
<div id="container">
<div id="target"></div>
</div>
<script>
document.body.offsetTop;
let target = document.querySelector("#target");
target.style.height = "auto";
test(() => {
assert_equals(window.getComputedStyle(target).height, "0px");
}, '#target height changed');
</script>
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment