Skip to content

Commit

Permalink
Expand definition of Invalid overflow rect to be zero-sized rects tha…
Browse files Browse the repository at this point in the history
…t are offset from origin as well.

`IsValidOverflowRect` was originally introduced to consider only zero-sized
rects that are located at origin to be invalid in Bug 1800907. However,
it doesn't make sense for a zero-sized rect located away from origin to
contribute to overflow.

Differential Revision: https://phabricator.services.mozilla.com/D234097

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1940938
gecko-commit: dd59e15ede57dde39ced180086733c9e6a1de2c6
gecko-reviewers: layout-reviewers, TYLin
  • Loading branch information
dshin-moz authored and moz-wptsync-bot committed Jan 16, 2025
1 parent 0828391 commit 9db3ea2
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions css/css-overflow/scrollable-overflow-empty-newline-span.html
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
<!DOCTYPE html>
<html class="reftest-wait">
<link rel="author" title="David Shin" href="dshin@mozilla.com">
<link rel="help" href="bugzilla.mozilla.org/show_bug.cgi?id=1940938">
<link rel="match" href="../reference/ref-filled-green-100px-square-only.html">
<p>Test passes if there is a filled green square.</p>
<div id="dut" style="overflow: hidden; height: 100px; width: 100px; background: red;">
<div style="width: 100px; height: 100px; background: green;"></div>
<div style="font-size: 200px;"><span>
</span></div>
</div>
<script>
onload = function () {
dut.scrollTop = 100;
document.documentElement.className = "";
}
</script>
</html>

0 comments on commit 9db3ea2

Please sign in to comment.