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 940 matching lines @@
                     23),  // (bottom left)  = max((78, 78) - (60, 55), (0, 0))
           FloatSize(6,
                     1)),  //  (bottom right) = max((56, 56) - (50, 55), (0, 0))
       borderRadiusClip->clipRect());
   EXPECT_EQ(frameContentClip(), borderRadiusClip->parent());
   CHECK_EXACT_VISUAL_RECT(LayoutRect(0, 0, 610, 500), &div,
                           document().view()->layoutView());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, TransformNodesAcrossSubframes) {
+  setChildFrameHTML(
+      "<style>body { margin: 0; }</style><div id='transform' style='transform: "
+      "translate3d(4px, 5px, 6px); width: 100px; height: 200px'></div>");
+
   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",
-      "<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());
 
-  LayoutObject* innerDivWithTransform =
-      frameDocument.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());
+    LayoutObject* innerDivWithTransform =
+        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.
-  // This asserts that we have the following tree structure:
-  // ...
-  //   Transform transform=translation=1.000000,2.000000,3.000000
-  //     PreTranslation transform=translation=7.000000,7.000000,0.000000
-  //       ScrollTranslation transform=translation=0.000000,0.000000,0.000000
-  //         Transform transform=translation=4.000000,5.000000,6.000000
-  auto* innerDocumentScrollTranslation = innerDivTransform->parent();
-  EXPECT_EQ(TransformationMatrix().translate3d(0, 0, 0),
-            innerDocumentScrollTranslation->matrix());
-  auto* iframePreTranslation = innerDocumentScrollTranslation->parent();
-  EXPECT_EQ(TransformationMatrix().translate3d(7, 7, 0),
-            iframePreTranslation->matrix());
-  EXPECT_EQ(divWithTransformProperties->transform(),
-            iframePreTranslation->parent());
+    // Ensure that the inner div's transform is correctly rooted in the root
+    // frame's transform tree.
+    // This asserts that we have the following tree structure:
+    // ...
+    //   Transform transform=translation=1.000000,2.000000,3.000000
+    //     PreTranslation transform=translation=7.000000,7.000000,0.000000
+    //       ScrollTranslation transform=translation=0.000000,0.000000,0.000000
+    //         Transform transform=translation=4.000000,5.000000,6.000000
+    auto* innerDocumentScrollTranslation = innerDivTransform->parent();
+    EXPECT_EQ(TransformationMatrix().translate3d(0, 0, 0),
+              innerDocumentScrollTranslation->matrix());
+    auto* iframePreTranslation = innerDocumentScrollTranslation->parent();
+    EXPECT_EQ(TransformationMatrix().translate3d(7, 7, 0),
+              iframePreTranslation->matrix());
+    EXPECT_EQ(divWithTransformProperties->transform(),
+              iframePreTranslation->parent());
 }
 
 TEST_P(PaintPropertyTreeBuilderTest, TransformNodesInTransformedSubframes) {
+  setChildFrameHTML(
+      "<style>body { margin: 31px; }</style><div "
+      "id='transform' style='transform: translate3d(7px, 8px, "
+      "9px); width: 100px; height: 200px'></div>");
   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>");
   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 1867 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