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

Bug 1588954 - Downgrade an assertion checking inline-size less than zero to a soft assertion. r=dholbert 

`nsIFrame::ComputeISizeValue` clamps the result to non-negative before
returning it, so the assertion doesn't need to be a hard assertion.

Differential Revision: https://phabricator.services.mozilla.com/D84745
This commit is contained in:
Ting-Yu Lin 2020-07-24 00:57:13 +00:00
Родитель eabaeb4c02
Коммит e0472f8fcc
4 изменённых файлов: 44 добавлений и 1 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

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

@ -6284,7 +6284,7 @@ nscoord nsIFrame::ComputeISizeValue(gfxContext* aRenderingContext,
"have unconstrained inline-size; this should only result from "
"very large sizes, not attempts at intrinsic inline-size "
"calculation");
MOZ_ASSERT(aContainingBlockISize >= 0, "inline-size less than zero");
NS_ASSERTION(aContainingBlockISize >= 0, "inline-size less than zero");
nscoord result = aCoord.Resolve(aContainingBlockISize);
// The result of a calc() expression might be less than 0; we

21
layout/reftests/bugs/1588954-ref.html Normal file
Просмотреть файл

@ -0,0 +1,21 @@
<!DOCTYPE html>
<style>
#multicol {
font: 16px/1 sans-serif;
height: 10px;
column-width: 30px;
column-fill: auto;
border: 1px solid gray;
}
.box {
writing-mode: vertical-rl;
width: 10px;
height: 20px;
border: 1px solid blue;
}
</style>
<div id="multicol">
<span>a</span>
<main class="box"></main>
</div>

21
layout/reftests/bugs/1588954.html Normal file
Просмотреть файл

@ -0,0 +1,21 @@
<!DOCTYPE html>
<style>
#multicol {
font: 16px/1 sans-serif;
max-height: 10px; /* Bound column height without using a fixed height. */
column-width: 30px;
column-fill: auto;
border: 1px solid gray;
}
.box {
writing-mode: vertical-rl;
width: 10px;
height: 20px;
border: 1px solid blue;
}
</style>
<div id="multicol">
<span>a</span>
<main class="box"></main>
</div>

1
layout/reftests/bugs/reftest.list
Просмотреть файл

@ -2092,6 +2092,7 @@ test-pref(plain_text.wrap_long_lines,false) != 1565129.txt 1565129.txt
fuzzy(0-32,0-8) fuzzy-if(Android&&webrender,0-32,0-1458) == 1576553-1.html 1576553-1-ref.html
fuzzy(0-1,0-10000) == 1577566-1.html 1577566-1-ref.html
== 1579953-2.html 1579953-2-ref.html
asserts(1-2) == 1588954.html 1588954-ref.html # Bug 1588954
== 1595199-1.html 1595199-1-ref.html
fuzzy-if(gtkWidget,0-255,0-42) == 1599841-1.html 1599841-1-ref.html
pref(layout.css.xul-box-display-values.content.enabled,true) == 1606130.html 1606130-ref.html