Bug 1242781 - Add 1 appunit in BRFrame::AddINlinePrefISize to match the 1 appunit added during reflow. r=dbaron

--HG--
extra : rebase_source : a8401f04f565e3e475e4a0ec19781b54ceb335c2
This commit is contained in:
Shing Lyu 2016-05-08 19:41:00 -04:00
Родитель 00f4a04098
Коммит 08c980d3a0
6 изменённых файлов: 26 добавлений и 3 удалений

Просмотреть файл

@ -139,6 +139,7 @@ BRFrame::Reflow(nsPresContext* aPresContext,
// XXX This also fixes bug 10036!
// Warning: nsTextControlFrame::CalculateSizeStandard depends on
// the following line, see bug 228752.
// The code below in AddInlinePrefISize also adds 1 appunit to width
finalSize.ISize(wm) = 1;
}
@ -178,6 +179,8 @@ BRFrame::AddInlinePrefISize(nsRenderingContext *aRenderingContext,
nsIFrame::InlinePrefISizeData *aData)
{
if (!GetParent()->StyleContext()->ShouldSuppressLineBreak()) {
// Match the 1 appunit width assigned in the Reflow method above
aData->mCurrentLine += 1;
aData->ForceBreak();
}
}

Просмотреть файл

@ -0,0 +1,6 @@
<!DOCTYPE HTML>
<html>
<body>
<p style="display:inline-block">some&nbsp;<span>te<br/>xt</span></p>
</body>
</html>

Просмотреть файл

@ -0,0 +1,6 @@
<!DOCTYPE HTML>
<html>
<body>
<p style="display:inline-block">some <span>te<br/>xt</span></p>
</body>
</html>

Просмотреть файл

@ -4,7 +4,14 @@
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<title>Testcase</title>
<style>
table[dir="rtl"] {background-color: #CCF}
table[dir="rtl"] {
background-color: #CCF;
/*
Add a left margin for the 1-appunit <br> hack in bug 1242781
The margin should be removed after bug 421436 is fixed
*/
margin-left: 0.016667px;
}
table[dir="ltr"] {background-color: #FCC}
</style>
</head>

Просмотреть файл

@ -1237,7 +1237,7 @@ fuzzy-if(skiaContent,1,12000) == 461512-1.html 461512-1-ref.html
skip-if((B2G&&browserIsRemote)||Mulet) == 463217-1.xul 463217-1-ref.xul # bug 974780 # Initial mulet triage: parity with B2G/B2G Desktop
skip-if(B2G||Mulet) == 463952-1.html 463952-1-ref.html # Initial mulet triage: parity with B2G/B2G Desktop
== 464811-1.html 464811-1-ref.html
fails == 465574-1.html 465574-1-ref.html # bug 421436
== 465574-1.html 465574-1-ref.html # bug 421436
== 466258-1.html 466258-1-ref.html
== 466395-1.html 466395-1-ref.html
== 466395-2.html 466395-2-ref.html
@ -1949,6 +1949,7 @@ random-if(OSX==1006) == 1238243-2.html 1238243-2-ref.html # fails on 10.6 with d
fuzzy(100,2000) == 1239564.html 1239564-ref.html
== 1242172-1.html 1242172-1-ref.html
== 1242172-2.html 1242172-2-ref.html
== 1242781.html 1242781-ref.html
== 1263845.html 1263845-ref.html
== 1260543-1.html 1260543-1-ref.html
== 1272997-1.html 1272997-1-ref.html

Просмотреть файл

@ -29,7 +29,7 @@ HTTP(..) == lineheight-metrics-2a.html lineheight-metrics-2-ref.html
HTTP(..) == lineheight-metrics-2b.html lineheight-metrics-2-ref.html
== lineheight-percentage-1.html lineheight-percentage-1-ref.html
skip-if(B2G||Mulet) == long-1.html long-ref.html # Initial mulet triage: parity with B2G/B2G Desktop
fuzzy-if(Android,255,42) == pre-line-1.html pre-line-1-ref.html
fuzzy-if(Android,255,147) == pre-line-1.html pre-line-1-ref.html
== pre-line-2.html pre-line-2-ref.html
== pre-line-3.html pre-line-3-ref.html
== pre-line-4.html pre-line-4-ref.html