Bug 1305579 - If we fail to restore the scroll position while there's a reflow pending, hold on to it and try again later. r=tnikkel

MozReview-Commit-ID: DNmFJOiNeeW

--HG--
extra : rebase_source : b44d4c04c76a21fe00d8ae460068359550d3d136
This commit is contained in:
Kartikaya Gupta 2016-09-27 14:40:56 -04:00
Родитель 3b2f21794b
Коммит 3ac8695431
3 изменённых файлов: 39 добавлений и 1 удалений

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

@ -4227,7 +4227,7 @@ ScrollFrameHelper::ScrollToRestoredPosition()
if (!weakFrame.IsAlive()) {
return;
}
if (PageIsStillLoading()) {
if (PageIsStillLoading() || NS_SUBTREE_DIRTY(mOuter)) {
// If we're trying to do a history scroll restore, then we want to
// keep trying this until we succeed, because the page can be loading
// incrementally. So re-get the scroll position for the next iteration,

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

@ -147,3 +147,4 @@ support-files = file_taintedfilters_feDisplacementMap-tainted-1.svg file_tainted
[test_scroll_position_restore.html]
support-files = file_scroll_position_restore.html
[test_scroll_animation_restore.html]
[test_scroll_position_iframe.html]

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

@ -0,0 +1,37 @@
<!DOCTYPE HTML>
<html>
<!--
https://bugzilla.mozilla.org/show_bug.cgi?id=1305579
-->
<head>
<meta charset="utf-8">
<title>Test for Bug 1305579</title>
<script type="application/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
<script type="text/javascript" src="/tests/SimpleTest/paint_listener.js"></script>
<link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/>
</head>
<body>
<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=1305579">Mozilla Bug 1305579</a>
<p id="display"></p>
<style>
.off {
display:none;
}
</style>
<script>
SimpleTest.waitForExplicitFinish();
function runtest() {
var iframe = document.getElementById("iframe");
iframe.contentDocument.scrollingElement.scrollTop = 50;
iframe.classList.toggle("off");
waitForAllPaintsFlushed(function() {
iframe.classList.toggle("off");
is(iframe.contentDocument.scrollingElement.scrollTop, 50, "scroll position restored");
SimpleTest.finish();
});
}
</script>
<iframe onload="runtest()" id="iframe" class="" src="data:text/html,<p>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br>bla<br></p>"></iframe><br>
</body>
</html>