mozilla
/
gecko-dev
зеркало из https://github.com/mozilla/gecko-dev.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

Bug 810726: Rework logic in nsColumnSetFrame for handling overflow to prevent infinite loop problems. [r=roc]

This commit is contained in:
Scott Johnson 2013-02-13 14:39:40 -06:00
Родитель 1966cf0df7
Коммит bd2aebb3c1
4 изменённых файлов: 34 добавлений и 7 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

8
layout/generic/crashtests/810726.html Normal file
Просмотреть файл

@ -0,0 +1,8 @@
<!DOCTYPE html>
<html>
<head>
</head>
<body>
<iframe width="1200" height="1024" src="810726-2.html">
</body>
</html>

21
layout/generic/crashtests/b810726-2.html Normal file
Просмотреть файл

@ -0,0 +1,21 @@
<!doctype html>
<html>
<head>
<style>
.outer-bound {
max-width: 980px;
}
.cols {
-moz-column-width: 700px;
}
</style>
</head>
<body>
<div class="outer-bound">
<div class="cols">
<p>There once was a guy.</p>
</div>
</div>
</body>
</html>

1
layout/generic/crashtests/crashtests.list
Просмотреть файл

@ -428,3 +428,4 @@ test-pref(layout.css.flexbox.enabled,true) load 798235-1.html
test-pref(layout.css.flexbox.enabled,true) load 799207-1.html
asserts(12) test-pref(layout.css.flexbox.enabled,true) load 799207-2.html
test-pref(layout.css.flexbox.enabled,true) load 804089-1.xhtml
load 810726.html

11
layout/generic/nsColumnSetFrame.cpp
Просмотреть файл

@ -735,7 +735,10 @@ nsColumnSetFrame::ReflowChildren(nsHTMLReflowMetrics& aDesiredSize,
}
if (columnCount >= aConfig.mBalanceColCount) {
if (contentBottom >= aReflowState.availableHeight) {
if (contentBottom > aReflowState.mComputedMaxHeight ||
contentBottom > aReflowState.ComputedHeight()) {
aColData.mShouldRevertToAuto = true;
} else {
// No more columns allowed here. Stop.
aStatus |= NS_FRAME_REFLOW_NEXTINFLOW;
kidNextInFlow->AddStateBits(NS_FRAME_IS_DIRTY);
@ -747,12 +750,6 @@ nsColumnSetFrame::ReflowChildren(nsHTMLReflowMetrics& aDesiredSize,
}
child = nullptr;
break;
} else if (contentBottom > aReflowState.mComputedMaxHeight ||
contentBottom > aReflowState.ComputedHeight()) {
aColData.mShouldRevertToAuto = true;
} else {
// The number of columns required is too high.
allFit = false;
}
}
}