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

Bug 550882. Make sure to create scrollbars for the viewport even if it's currently overflow:hidden, since various changes to <body> and the like can change the propagated overflow. r=roc

This commit is contained in:
Boris Zbarsky 2010-03-08 19:30:02 -05:00
Родитель 7569307a1d
Коммит a1af86a13d
7 изменённых файлов: 52 добавлений и 20 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

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

@ -2175,26 +2175,24 @@ nsGfxScrollFrameInner::CreateAnonymousContent(nsTArray<nsIContent*>& aElements)
nsIScrollableFrame *scrollable = do_QueryFrame(mOuter);
// At this stage in frame construction, the document element and/or
// BODY overflow styles have not yet been propagated to the
// viewport. So GetScrollbarStylesFromFrame called here will only
// take into account the scrollbar preferences set on the docshell.
// Thus if no scrollbar preferences are set on the docshell, we will
// always create scrollbars, which means later dynamic changes to
// propagated overflow styles will show or hide scrollbars on the
// viewport without requiring frame reconstruction of the viewport
// (good!).
// XXX On the other hand, if scrolling="no" is set on the container
// we won't create scrollbars here so no scrollbars will ever be
// created even if the container's scrolling attribute is later
// changed. However, this has never been supported.
ScrollbarStyles styles = scrollable->GetScrollbarStyles();
PRBool canHaveHorizontal = styles.mHorizontal != NS_STYLE_OVERFLOW_HIDDEN;
PRBool canHaveVertical = styles.mVertical != NS_STYLE_OVERFLOW_HIDDEN;
if (!canHaveHorizontal && !canHaveVertical) {
// Nothing to do.
return NS_OK;
// If we're the scrollframe for the root, then we want to construct
// our scrollbar frames no matter what. That way later dynamic
// changes to propagated overflow styles will show or hide
// scrollbars on the viewport without requiring frame reconstruction
// of the viewport (good!).
PRBool canHaveHorizontal;
PRBool canHaveVertical;
if (!mIsRoot) {
ScrollbarStyles styles = scrollable->GetScrollbarStyles();
canHaveHorizontal = styles.mHorizontal != NS_STYLE_OVERFLOW_HIDDEN;
canHaveVertical = styles.mVertical != NS_STYLE_OVERFLOW_HIDDEN;
if (!canHaveHorizontal && !canHaveVertical) {
// Nothing to do.
return NS_OK;
}
} else {
canHaveHorizontal = PR_TRUE;
canHaveVertical = PR_TRUE;
}
// The anonymous <div> used by <inputs> never gets scrollbars.

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

@ -0,0 +1,2 @@
<!DOCTYPE html>
<iframe src="data:text/html,<html></html>"></iframe>

8
layout/reftests/bugs/502447-2.html Normal file
Просмотреть файл

@ -0,0 +1,8 @@
<!DOCTYPE html>
<iframe src="data:text/html,<html style='height: 100%; overflow: visible'>
<body style='margin: 0; padding: 0; height: 100%'
onload='document.body.offsetWidth;
document.documentElement.style.overflow=&quot;hidden&quot;'>
<div style='width: 200%; height: 200%'></div>
</body>
</html>"></iframe>

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

@ -0,0 +1,6 @@
<!DOCTYPE html>
<iframe src="data:text/html,<html style='height: 100%'>
<body style='margin: 0; padding: 0; height: 100%'>
<div style='width: 200%; height: 200%'></div>
</body>
</html>"></iframe>

8
layout/reftests/bugs/550882-1.html Normal file
Просмотреть файл

@ -0,0 +1,8 @@
<!DOCTYPE html>
<iframe src="data:text/html,<html style='height: 100%'>
<body style='margin: 0; padding: 0; height: 100%; overflow: hidden'
onload='document.body.offsetWidth;
document.body.style.overflow=&quot;&quot;'>
<div style='width: 200%; height: 200%'></div>
</body>
</html>"></iframe>

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

@ -0,0 +1,2 @@
<!DOCTYPE html>
<iframe src="data:text/html,<html></html>"></iframe>

8
layout/reftests/bugs/550882-2.html Normal file
Просмотреть файл

@ -0,0 +1,8 @@
<!DOCTYPE html>
<iframe src="data:text/html,<html style='height: 100%'>
<body style='margin: 0; padding: 0; height: 100%; overflow: visible'
onload='document.body.offsetWidth;
document.body.style.overflow=&quot;hidden&quot;'>
<div style='width: 200%; height: 200%'></div>
</body>
</html>"></iframe>