Commit 28ebe55f authored by Vladimir Levin's avatar Vladimir Levin Committed by Commit Bot

[PE]: Remove LayoutViewIsolationNodes flag which has been on for a while.

This patch removes the flag added earlier, since the feature has been
on without any problems.

R=chrishtr@chromium.org, pdr@chromium.org

Change-Id: I42e41417f531e2ba47a096cf172234d4f549a5fd
Reviewed-on: https://chromium-review.googlesource.com/c/1394915Reviewed-by: default avatarChris Harrelson <chrishtr@chromium.org>
Reviewed-by: default avatarMason Freed <masonfreed@chromium.org>
Commit-Queue: vmpstr <vmpstr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#619963}
parent dcec81e1
...@@ -322,8 +322,7 @@ static bool NeedsIsolationNodes(const LayoutObject& object) { ...@@ -322,8 +322,7 @@ static bool NeedsIsolationNodes(const LayoutObject& object) {
// Layout view establishes isolation with the exception of local roots (since // Layout view establishes isolation with the exception of local roots (since
// they are already essentially isolated). // they are already essentially isolated).
if (RuntimeEnabledFeatures::LayoutViewIsolationNodesEnabled() && if (object.IsLayoutView()) {
object.IsLayoutView()) {
const auto* parent_frame = object.GetFrame()->Tree().Parent(); const auto* parent_frame = object.GetFrame()->Tree().Parent();
return parent_frame && parent_frame->IsLocalFrame(); return parent_frame && parent_frame->IsLocalFrame();
} }
......
...@@ -1857,9 +1857,6 @@ TEST_P(PaintPropertyTreeBuilderTest, TransformNodesAcrossSubframes) { ...@@ -1857,9 +1857,6 @@ TEST_P(PaintPropertyTreeBuilderTest, TransformNodesAcrossSubframes) {
} }
TEST_P(PaintPropertyTreeBuilderTest, FramesEstablishIsolation) { TEST_P(PaintPropertyTreeBuilderTest, FramesEstablishIsolation) {
if (!RuntimeEnabledFeatures::LayoutViewIsolationNodesEnabled())
return;
SetBodyInnerHTML(R"HTML( SetBodyInnerHTML(R"HTML(
<style> <style>
body { margin: 0; } body { margin: 0; }
......
...@@ -694,10 +694,6 @@ ...@@ -694,10 +694,6 @@
name: "LayoutNGFragmentCaching", name: "LayoutNGFragmentCaching",
implied_by: ["LayoutNG"], implied_by: ["LayoutNG"],
}, },
{
name: "LayoutViewIsolationNodes",
status: "stable",
},
{ {
name: "LazyFrameLoading", name: "LazyFrameLoading",
}, },
......
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