Backed out changeset 1b29d7a6d15c (bug 1718012) for causing crashtest failures. CLOSED TREE

This commit is contained in:
Sandor Molnar 2021-07-28 09:24:58 +03:00
parent 551ded00a4
commit 307d5a322a
5 changed files with 8 additions and 42 deletions

View File

@ -1,36 +0,0 @@
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>Test for Bug 1223561</title>
<script src="/tests/SimpleTest/EventUtils.js"></script>
<script src="/tests/SimpleTest/SimpleTest.js"></script>
<script src="/tests/SimpleTest/WindowSnapshot.js"></script>
<script type="text/javascript" src="file_fullscreen-utils.js"></script>
</head>
<body>
<div id="target" style="width: 100px; height: 100px; background-color: green;"></div>
<script>
"use strict";
function begin() {
info("Setting full zoom to 30%");
SpecialPowers.setFullZoom(window, 0.3);
addFullscreenChangeContinuation("enter", enteredFullscreen);
target.requestFullscreen();
}
function enteredFullscreen() {
info("The element with green background should be in fullscreen");
assertWindowPureColor(window, "green");
addFullscreenChangeContinuation("exit", exitedFullscreen);
document.exitFullscreen();
}
function exitedFullscreen() {
opener.nextTest();
}
</script>
</body>
</html>

View File

@ -471,7 +471,6 @@ support-files =
file_fullscreen-table.html
file_fullscreen-top-layer.html
file_fullscreen-utils.js
file_fullscreen-with-full-zoom.html
[test_fullscreen_meta_viewport.html]
support-files = file_fullscreen_meta_viewport.html

View File

@ -54,7 +54,6 @@ var gTestWindows = [
["dom.security.featurePolicy.webidl.enabled", true]] },
{ test: "file_fullscreen-async.html" },
{ test: "file_fullscreen-sub-iframe.html" },
{ test: "file_fullscreen-with-full-zoom.html" },
];
var testWindow = null;

View File

@ -566,6 +566,10 @@ void MobileViewportManager::RefreshVisualViewportSize() {
void MobileViewportManager::UpdateSizesBeforeReflow() {
if (Maybe<LayoutDeviceIntSize> newDisplaySize =
mContext->GetContentViewerSize()) {
if (mDisplaySize == *newDisplaySize) {
return;
}
mDisplaySize = *newDisplaySize;
MVM_LOG("%p: Reflow starting, display size updated to %s\n", this,
ToString(mDisplaySize).c_str());

View File

@ -2009,10 +2009,6 @@ void PresShell::SimpleResizeReflow(nscoord aWidth, nscoord aHeight,
FrameNeedsReflow(rootFrame, IntrinsicDirty::Resize,
NS_FRAME_HAS_DIRTY_CHILDREN);
if (mMobileViewportManager) {
mMobileViewportManager->UpdateSizesBeforeReflow();
}
// For compat with the old code path which always reflowed as long as there
// was a root frame.
bool suppressReflow = (aOptions & ResizeReflowOptions::SuppressReflow) ||
@ -9492,6 +9488,10 @@ bool PresShell::DoReflow(nsIFrame* target, bool aInterruptible,
timeStart = TimeStamp::Now();
}
if (mMobileViewportManager) {
mMobileViewportManager->UpdateSizesBeforeReflow();
}
// Schedule a paint, but don't actually mark this frame as changed for
// retained DL building purposes. If any child frames get moved, then
// they will schedule paint again. We could probaby skip this, and just