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

Bug 395331. Fix FindPrimaryFrameFor by noticing that an element can be in anonymous content if the binding root is a binding *ancestor*, not just a binding *parent*. r+sr=bzbarsy

This commit is contained in:
roc+@cs.cmu.edu 2007-10-07 19:42:50 -07:00
Родитель 9f28173b95
Коммит 7b135748b0
5 изменённых файлов: 73 добавлений и 1 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

17
layout/base/nsCSSFrameConstructor.cpp
Просмотреть файл

@ -10644,6 +10644,21 @@ nsCSSFrameConstructor::ReplicateFixedFrames(nsPageContentFrame* aParentFrame)
return NS_OK;
}
static PRBool
IsBindingAncestor(nsIContent* aContent, nsIContent* aBindingRoot)
{
while (PR_TRUE) {
nsIContent* bindingParent = aContent->GetBindingParent();
if (!bindingParent)
return PR_FALSE;
if (bindingParent == aBindingRoot)
return PR_TRUE;
if (bindingParent == aContent)
return PR_FALSE;
aContent = bindingParent;
}
}
// Helper function that searches the immediate child frames
// (and their children if the frames are "special")
// for a frame that maps the specified content object
@ -10738,7 +10753,7 @@ nsCSSFrameConstructor::FindFrameWithContent(nsFrameManager* aFrameManager,
// We also need to search if the child content is anonymous and scoped
// to the parent content.
if (aParentContent == kidContent ||
(aParentContent && (aParentContent == kidContent->GetBindingParent())))
(aParentContent && IsBindingAncestor(kidContent, aParentContent)))
{
#ifdef NOISY_FINDFRAME
FFWC_recursions++;

6
layout/reftests/bugs/395331-1-ref.xml Normal file
Просмотреть файл

@ -0,0 +1,6 @@
<html xmlns="http://www.w3.org/1999/xhtml"
xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
<body>
Kitty
</body>
</html>

20
layout/reftests/bugs/395331-1.xml Normal file
Просмотреть файл

@ -0,0 +1,20 @@
<html xmlns="http://www.w3.org/1999/xhtml"
xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul">
<head>
<style>
.base { -moz-binding: url("395331-xbl.xml#base"); }
.inner { -moz-binding: url("395331-xbl.xml#inner"); }
</style>
<script>
function boom()
{
var x = document.getElementById("x");
x.removeChild(x.firstChild);
}
</script>
</head>
<body onload="boom();">
<xul:box class="base" id="x" style="height:2em;">Hello</xul:box>
Kitty
</body>
</html>

30
layout/reftests/bugs/395331-xbl.xml Normal file
Просмотреть файл

@ -0,0 +1,30 @@
<?xml version="1.0"?>
<!DOCTYPE bindings>
<bindings id="testBindings"
xmlns="http://www.mozilla.org/xbl"
xmlns:xul="http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul"
xmlns:xbl="http://www.mozilla.org/xbl">
<binding id="base">
<content>
<xul:box flex="1">
<xul:box class="inner" flex="1">
<children/>
</xul:box>
</xul:box>
</content>
</binding>
<binding id="inner">
<content>
<xul:box flex="1">
<xul:box flex="1">
<children/>
</xul:box>
</xul:box>
</content>
</binding>
</bindings>

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

@ -397,5 +397,6 @@ fails == 386310-1d.html 386310-1-ref.html
== 394676-1.xhtml 394676-1-ref.xhtml
== 395130-1.html 395130-1-ref.html
== 395130-2.html 395130-2-ref.html
== 395331-1.xml 395331-1-ref.xml
== 396286-1.html about:blank # crash test
== 398289-1.html 398289-1-ref.html