Don't call isURLAllowed() from layout.

third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "core/layout/LayoutTestHelper.h"
 #include "core/layout/LayoutTreeAsText.h"
 #include "core/layout/api/LayoutViewItem.h"
 #include "core/paint/ObjectPaintProperties.h"
 #include "core/paint/PaintPropertyTreePrinter.h"
 #include "platform/graphics/paint/GeometryMapper.h"
@@ skipping 256 matching lines @@
       absPos->layoutObject()->paintProperties();
   EXPECT_EQ(TransformationMatrix().translate(123, 456),
             absPosProperties->paintOffsetTranslation()->matrix());
   EXPECT_EQ(frameScrollTranslation(),
             absPosProperties->paintOffsetTranslation()->parent());
   EXPECT_EQ(absPosProperties->transform(),
             absPosProperties->overflowClip()->localTransformSpace());
   EXPECT_EQ(FloatRoundedRect(0, 0, 300, 400),
             absPosProperties->overflowClip()->clipRect());
   EXPECT_EQ(frameContentClip(), absPosProperties->overflowClip()->parent());
-    CHECK_EXACT_VISUAL_RECT(LayoutRect(123, 456, 300, 400),
-                            absPos->layoutObject(), frameView->layoutView());
+  CHECK_EXACT_VISUAL_RECT(LayoutRect(123, 456, 300, 400),
+                          absPos->layoutObject(), frameView->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, FrameScrollingTraditional) {
   setBodyInnerHTML("<style> body { height: 10000px; } </style>");
 
   document().domWindow()->scrollTo(0, 100);
 
   FrameView* frameView = document().view();
   frameView->updateAllLifecyclePhases();
   EXPECT_EQ(TransformationMatrix(), framePreTranslation()->matrix());
@@ skipping 52 matching lines @@
   EXPECT_EQ(TransformationMatrix().translate3d(123, 456, 789),
             transformProperties->transform()->matrix());
   EXPECT_EQ(FloatPoint3D(200, 150, 0),
             transformProperties->transform()->origin());
   EXPECT_EQ(transformProperties->paintOffsetTranslation(),
             transformProperties->transform()->parent());
   EXPECT_EQ(TransformationMatrix().translate(50, 100),
             transformProperties->paintOffsetTranslation()->matrix());
   EXPECT_EQ(frameScrollTranslation(),
             transformProperties->paintOffsetTranslation()->parent());
-    CHECK_EXACT_VISUAL_RECT(LayoutRect(173, 556, 400, 300),
-                            transform->layoutObject(),
-                            document().view()->layoutView());
+  CHECK_EXACT_VISUAL_RECT(LayoutRect(173, 556, 400, 300),
+                          transform->layoutObject(),
+                          document().view()->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, RelativePositionInline) {
   loadTestData("relative-position-inline.html");
 
   Element* inlineBlock = document().getElementById("inline-block");
   const ObjectPaintProperties* inlineBlockProperties =
       inlineBlock->layoutObject()->paintProperties();
   EXPECT_EQ(TransformationMatrix().translate(135, 490),
             inlineBlockProperties->paintOffsetTranslation()->matrix());
@@ skipping 591 matching lines @@
   EXPECT_EQ(frameContentClip(), borderRadiusClip->parent());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 610, 500), &div,
                           document().view()->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, TransformNodesAcrossSubframes) {
   setBodyInnerHTML(
       "<style>body { margin: 0; }</style>"
       "<div id='divWithTransform' style='transform: translate3d(1px, 2px, "
       "3px);'>"
-      "  <iframe style='border: 7px solid black' id='frame'></iframe>"
+      "  <iframe style='border: 7px solid black'></iframe>"
       "</div>");
-  Document& frameDocument = setupChildIframe(
-      "frame",
+  setChildFrameHTML(
       "<style>body { margin: 0; }</style><div id='transform' style='transform: "
       "translate3d(4px, 5px, 6px); width: 100px; height: 200px'></div>");
+
   FrameView* frameView = document().view();
   frameView->updateAllLifecyclePhases();
 
   LayoutObject* divWithTransform =
       document().getElementById("divWithTransform")->layoutObject();
   const ObjectPaintProperties* divWithTransformProperties =
       divWithTransform->paintProperties();
   EXPECT_EQ(TransformationMatrix().translate3d(1, 2, 3),
             divWithTransformProperties->transform()->matrix());
-    CHECK_EXACT_VISUAL_RECT(LayoutRect(1, 2, 800, 164), divWithTransform,
-                            frameView->layoutView());
+  CHECK_EXACT_VISUAL_RECT(LayoutRect(1, 2, 800, 164), divWithTransform,
+                          frameView->layoutView());
 
   LayoutObject* innerDivWithTransform =
-      frameDocument.getElementById("transform")->layoutObject();
+      childDocument().getElementById("transform")->layoutObject();
   const ObjectPaintProperties* innerDivWithTransformProperties =
       innerDivWithTransform->paintProperties();
   auto* innerDivTransform = innerDivWithTransformProperties->transform();
   EXPECT_EQ(TransformationMatrix().translate3d(4, 5, 6),
             innerDivTransform->matrix());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(12, 14, 100, 145), innerDivWithTransform,
                           frameView->layoutView());
 
   // Ensure that the inner div's transform is correctly rooted in the root
   // frame's transform tree.
@@ skipping 11 matching lines @@
             iframePreTranslation->matrix());
   EXPECT_EQ(divWithTransformProperties->transform(),
             iframePreTranslation->parent());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, TransformNodesInTransformedSubframes) {
   setBodyInnerHTML(
       "<style>body { margin: 0; }</style>"
       "<div id='divWithTransform' style='transform: translate3d(1px, 2px, "
       "3px);'>"
-      "  <iframe id='frame' style='transform: translate3d(4px, 5px, 6px); "
+      "  <iframe style='transform: translate3d(4px, 5px, 6px); "
       "border: 42px solid; margin: 7px;'></iframe>"
       "</div>");
-  Document& frameDocument =
-      setupChildIframe("frame",
-                       "<style>body { margin: 31px; }</style><div "
-                       "id='transform' style='transform: translate3d(7px, 8px, "
-                       "9px); width: 100px; height: 200px'></div>");
+  setChildFrameHTML(
+      "<style>body { margin: 31px; }</style><div "
+      "id='transform' style='transform: translate3d(7px, 8px, "
+      "9px); width: 100px; height: 200px'></div>");
   FrameView* frameView = document().view();
   frameView->updateAllLifecyclePhases();
 
   // Assert that we have the following tree structure:
   // ...
   //   Transform transform=translation=1.000000,2.000000,3.000000
   //     PaintOffsetTranslation transform=translation=7.000000,7.000000,0.000000
   //       Transform transform=translation=4.000000,5.000000,6.000000
   //         PreTranslation transform=translation=42.000000,42.000000,0.000000
   //           ScrollTranslation transform=translation=0.000000,0.000000,0.00000
   //             PaintOffsetTranslation transform=translation=31.00,31.00,0.00
   //               Transform transform=translation=7.000000,8.000000,9.000000
 
   LayoutObject* innerDivWithTransform =
-      frameDocument.getElementById("transform")->layoutObject();
+      childDocument().getElementById("transform")->layoutObject();
   auto* innerDivTransform =
       innerDivWithTransform->paintProperties()->transform();
   EXPECT_EQ(TransformationMatrix().translate3d(7, 8, 9),
             innerDivTransform->matrix());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(92, 95, 100, 111), innerDivWithTransform,
                           frameView->layoutView());
 
   auto* innerDocumentPaintOffsetTranslation = innerDivTransform->parent();
   EXPECT_EQ(TransformationMatrix().translate3d(31, 31, 0),
             innerDocumentPaintOffsetTranslation->matrix());
@@ skipping 11 matching lines @@
   EXPECT_EQ(TransformationMatrix().translate3d(7, 7, 0),
             iframePaintOffsetTranslation->matrix());
   auto* divWithTransformTransform = iframePaintOffsetTranslation->parent();
   EXPECT_EQ(TransformationMatrix().translate3d(1, 2, 3),
             divWithTransformTransform->matrix());
 
   LayoutObject* divWithTransform =
       document().getElementById("divWithTransform")->layoutObject();
   EXPECT_EQ(divWithTransformTransform,
             divWithTransform->paintProperties()->transform());
-    CHECK_EXACT_VISUAL_RECT(LayoutRect(1, 2, 800, 248), divWithTransform,
-                            frameView->layoutView());
+  CHECK_EXACT_VISUAL_RECT(LayoutRect(1, 2, 800, 248), divWithTransform,
+                          frameView->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, TreeContextClipByNonStackingContext) {
   // This test verifies the tree builder correctly computes and records the
   // property tree context for a (pseudo) stacking context that is scrolled by a
   // containing block that is not one of the painting ancestors.
   setBodyInnerHTML(
       "<style>body { margin: 0; }</style>"
       "<div id='scroller' style='overflow:scroll; width:400px; height:300px;'>"
       "  <div id='child' style='position:relative; width:100px; height: "
@@ skipping 46 matching lines @@
 
   EXPECT_EQ(
       frameContentClip(),
       childProperties->localBorderBoxProperties()->propertyTreeState.clip());
   EXPECT_EQ(frameScrollTranslation(),
             childProperties->localBorderBoxProperties()
                 ->propertyTreeState.transform());
   EXPECT_EQ(
       scrollerProperties->effect(),
       childProperties->localBorderBoxProperties()->propertyTreeState.effect());
-    CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 800, 10000), &scroller,
-                            document().view()->layoutView());
+  CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 800, 10000), &scroller,
+                          document().view()->layoutView());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 100, 200), &child,
                           document().view()->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, TableCellLayoutLocation) {
   // This test verifies that the border box space of a table cell is being
   // correctly computed.  Table cells have weird location adjustment in our
   // layout/paint implementation.
   setBodyInnerHTML(
       "<style>"
@@ skipping 1770 matching lines @@
   EXPECT_EQ(framePreTranslation(),
             childProperties->paintOffsetTranslation()->parent());
   EXPECT_EQ(childProperties->paintOffsetTranslation(),
             childPaintState.transform());
   // This will change once we added clip expansion node.
   EXPECT_EQ(filterProperties->effect()->outputClip(), childPaintState.clip());
   EXPECT_EQ(filterProperties->effect(), childPaintState.effect());
 }
 
 }  // namespace blink