Skip to content

GitLab

T
tangled
Commit c52cf1af authored by Koji Ishii's avatar Koji Ishii Committed by Commit Bot
Browse files
Options

Add tests for atomic inline+NBSP quirk in nowrap 

This patch adds a test to check the atomic inline+NBSP
quirk[1] should not apply if `nowrap`.

This issue was originally introduced in r694788
crrev.com/c/1792465, but was accidentally fixed in r700578
crrev.com/c/1827097. The fix did not have a test because
it wasn't intended to fix this issue.

[1] https://www.w3.org/TR/css-text-3/#line-break-details

TBR=chrishtr@chromium.org, ikilpatrick@chromium.org

Bug: 1017465, 1018309
Change-Id: Id505b894c02d6521766e2cf354e7c9c8c4a114ce
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1888230Reviewed-by: default avatarIan Kilpatrick <ikilpatrick@chromium.org>
Reviewed-by: default avatarKoji Ishii <kojii@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#712447}
parent eeb9d035
Hide whitespace changes
Inline Side-by-side
Showing with 40 additions and 0 deletions
third_party/blink/web_tests/external/wpt/css/css-text/line-breaking/line-breaking-atomic-nowrap-001.html 0 → 100644
View file @ c52cf1af
<!DOCTYPE html>
<title>CSS Text Test: Check atomic inline+NBSP quirk does not apply if nowrap</title>
<link rel="help" href="https://www.w3.org/TR/css-text-3/#line-break-details">
<link rel="help" href="https://crbug.com/1002442">
<link rel="author" title="Koji Ishii" href="mailto:kojiishi@gmail.com">
<script src="/resources/testharness.js"></script>
<script src="/resources/testharnessreport.js"></script>
<style>
div {
white-space: nowrap;
font-size: 10px;
line-height: 1;
width: 5ch;
}
.inline-block {
display: inline-block;
width: 3ch;
height: 1em;
}
img {
width: 3ch;
height: 1em;
}
</style>
<body>
<div>000<span class="inline-block"></span>&nbsp;111</div>
<div>000<img>&nbsp;111</div>
<div>000&nbsp;<span class="inline-block"></span>111</div>
<div>000&nbsp;<img>111</div>
<script>
(function () {
for (let e of document.getElementsByTagName('div')) {
test(() => {
let height = e.offsetHeight;
assert_equals(Math.round(height / 10), 1);
});
}
})();
</script>
</body>
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