Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(114)

Unified Diff: LayoutTests/fast/repaint/svg-layout-root-style-attr-update.html

Issue 933953003: Move the remaining rendering/svg/RenderSVG* files to layout. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: LayoutTests/fast/repaint/svg-layout-root-style-attr-update.html
diff --git a/LayoutTests/fast/repaint/svg-layout-root-style-attr-update.html b/LayoutTests/fast/repaint/svg-layout-root-style-attr-update.html
index 0c4049aefdaef72e36b1b1fd9822ad555d1c3daf..9fc96bf4684040ed39a272c06e40d2b06fb5b8d7 100644
--- a/LayoutTests/fast/repaint/svg-layout-root-style-attr-update.html
+++ b/LayoutTests/fast/repaint/svg-layout-root-style-attr-update.html
@@ -36,22 +36,22 @@ function test() {
greenRect.setAttribute("height", "50%");
// make sure the svg root's .style attribute is out of date
svgroot.style.position = "absolute";
- // force layout. this will be rooted at the RenderSVGRoot and will set m_posChildNeedsLayout on its
+ // force layout. this will be rooted at the LayoutSVGRoot and will set m_posChildNeedsLayout on its
// containing RenderBlock (corresponding to DIV#root)
document.body.offsetWidth;
- // dirty an SVG attribute, will set FrameView::m_layoutRoot to the RenderSVGRoot
+ // dirty an SVG attribute, will set FrameView::m_layoutRoot to the LayoutSVGRoot
greenRect.setAttribute("width", "50%");
// dirty a normal DOM attribute in a separate part of the DOM. this is where things go awry since
// FrameView::scheduleRelayoutOfSubtree will clear out its m_layoutRoot and call
- // LayoutObject::markContainingBlocksForLayout() on the RenderSVGRoot. Since the RenderSVGRoot's
- // container already has m_posChildNeedsLayout set, the RenderSVGRoot's container's container
+ // LayoutObject::markContainingBlocksForLayout() on the LayoutSVGRoot. Since the LayoutSVGRoot's
+ // container already has m_posChildNeedsLayout set, the LayoutSVGRoot's container's container
// (corresponding to the DIV#grandparent) will not have any needs layout flags set on it.
document.getElementById('other').style.width="500px";
// Run a layout pass. This will propagate the render tree up to the DIV#other's render object but
// will not traverse into the svg subtree at all since the DIV#grandparent's render object is
// not marked as needing layout.
document.body.offsetWidth;
- // This goes into the void since the RenderSVGRoot is already marked as needsLayout but there is no
+ // This goes into the void since the LayoutSVGRoot is already marked as needsLayout but there is no
// layout pending.
greenRect.setAttribute("x", "50%");
}

Powered by Google App Engine
This is Rietveld 408576698