wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 393655: Fix undetected cases of lines going empty --> nonempty, due to margins. r+sr=roc, a=blocking1.9+

Browse Source
This commit is contained in:
dholbert@cs.stanford.edu 2007-11-15 11:07:04 -08:00
parent 2c139cef8f
commit e042a678b0
8 changed files with 84 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
layout/generic/nsBlockFrame.cpp
View File

@ -1901,7 +1901,7 @@ nsBlockFrame::ReflowDirtyLines(nsBlockReflowState& aState)
// it wasn't empty before, any adjacency and clearance changes are irrelevant
// to the result of nextLine->ShouldApplyTopMargin.
if (line.next() != end_lines()) {
PRBool maybeWasEmpty = oldY == oldYMost;
PRBool maybeWasEmpty = oldY == line.next()->mBounds.y;
PRBool isEmpty = line->mBounds.height == 0 && line->CachedIsEmpty();
if (maybeReflowingForFirstTime /*1*/ ||
(isEmpty || maybeWasEmpty) /*2/3/4*/) {

9
layout/reftests/bugs/393655-3-ref.html Normal file
View File

@ -0,0 +1,9 @@
<html>
<body>
Foo
<div id="editMe" style="margin-bottom: 50px;">
a
</div>
<div>Bar</div>
</body>
</html>

17
layout/reftests/bugs/393655-3.html Normal file
View File

@ -0,0 +1,17 @@
<html class="reftest-wait">
<head>
<script>
function tweak() {
var editMe = document.getElementById('editMe');
editMe.innerHTML="a";
document.documentElement.className = "";
}
</script>
</head>
<body onload="tweak()">
Foo
<div id="editMe" style="margin-bottom: 50px;">
</div>
<div>Bar</div>
</body>
</html>

9
layout/reftests/bugs/393655-4-ref.html Normal file
View File

@ -0,0 +1,9 @@
<html>
<body>
Foo
<div id="editMe" style="margin-top: 50px;">
a
</div>
<div>Bar</div>
</body>
</html>

17
layout/reftests/bugs/393655-4.html Normal file
View File

@ -0,0 +1,17 @@
<html class="reftest-wait">
<head>
<script>
function tweak() {
var editMe = document.getElementById('editMe');
editMe.innerHTML="a";
document.documentElement.className = "";
}
</script>
</head>
<body onload="tweak()">
Foo
<div id="editMe" style="margin-top: 50px;">
</div>
<div>Bar</div>
</body>
</html>

9
layout/reftests/bugs/393655-5-ref.html Normal file
View File

@ -0,0 +1,9 @@
<html>
<body>
Foo
<div id="editMe" style="margin-bottom: 50px; margin-top: 50px;">
a
</div>
<div>Bar</div>
</body>
</html>

17
layout/reftests/bugs/393655-5.html Normal file
View File

@ -0,0 +1,17 @@
<html class="reftest-wait">
<head>
<script>
function tweak() {
var editMe = document.getElementById('editMe');
editMe.innerHTML="a";
document.documentElement.className = "";
}
</script>
</head>
<body onload="tweak()">
Foo
<div id="editMe" style="margin-bottom: 50px; margin-top: 50px;">
</div>
<div>Bar</div>
</body>
</html>

7
layout/reftests/bugs/reftest.list
View File

@ -418,8 +418,11 @@ fails == 386310-1d.html 386310-1-ref.html
== 393330-1.html 393330-1-ref.html
== 393517-1.xhtml about:blank # crash test
== 393649-1.html 393649-1-ref.html
fails == 393655-1.html 393655-1-ref.html # Remove 'fails' when 393655 lands
fails == 393655-2.html 393655-2-ref.html # Remove 'fails' when 393655 lands
== 393655-1.html 393655-1-ref.html
== 393655-2.html 393655-2-ref.html
== 393655-3.html 393655-3-ref.html
== 393655-4.html 393655-4-ref.html
== 393655-5.html 393655-5-ref.html
== 393671-1.html 393671-1-ref.html
== 393671-2.html 393671-2-ref.html
== 393671-3.html 393671-3-ref.html