Commit bd1646e5 authored by Dave Tapuska's avatar Dave Tapuska Committed by Commit Bot

Return DispayNoneIFrameCreatesNoLayoutObject.

Initially shipped in M65 this feature flag can be removed.

BUG=650433

Change-Id: I2088238be14ec263e2bb81dc0cb9730438315cb2
Reviewed-on: https://chromium-review.googlesource.com/c/1323891Reviewed-by: default avatarErik Chen <erikchen@chromium.org>
Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
Cr-Commit-Position: refs/heads/master@{#606110}
parent b47af5d5
...@@ -5296,11 +5296,9 @@ void Document::WillChangeFrameOwnerProperties(int margin_width, ...@@ -5296,11 +5296,9 @@ void Document::WillChangeFrameOwnerProperties(int margin_width,
DCHECK(GetFrame() && GetFrame()->Owner()); DCHECK(GetFrame() && GetFrame()->Owner());
FrameOwner* owner = GetFrame()->Owner(); FrameOwner* owner = GetFrame()->Owner();
if (RuntimeEnabledFeatures::DisplayNoneIFrameCreatesNoLayoutObjectEnabled()) { if (documentElement()) {
if (documentElement()) { if (is_display_none != owner->IsDisplayNone())
if (is_display_none != owner->IsDisplayNone()) documentElement()->LazyReattachIfAttached();
documentElement()->LazyReattachIfAttached();
}
} }
// body() may become null as a result of modification event listeners, so we // body() may become null as a result of modification event listeners, so we
......
...@@ -229,8 +229,6 @@ bool LayoutView::CanHaveChildren() const { ...@@ -229,8 +229,6 @@ bool LayoutView::CanHaveChildren() const {
FrameOwner* owner = GetFrame()->Owner(); FrameOwner* owner = GetFrame()->Owner();
if (!owner) if (!owner)
return true; return true;
if (!RuntimeEnabledFeatures::DisplayNoneIFrameCreatesNoLayoutObjectEnabled())
return true;
// Although it is not spec compliant, many websites intentionally call // Although it is not spec compliant, many websites intentionally call
// Window.print() on display:none iframes. https://crbug.com/819327. // Window.print() on display:none iframes. https://crbug.com/819327.
if (GetDocument().Printing()) if (GetDocument().Printing())
......
...@@ -391,10 +391,6 @@ ...@@ -391,10 +391,6 @@
{ {
name: "DisplayLocking", name: "DisplayLocking",
}, },
{
name: "DisplayNoneIFrameCreatesNoLayoutObject",
status: "stable",
},
{ {
name: "DocumentCookie", name: "DocumentCookie",
}, },
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment