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

Side by Side Diff: Source/core/rendering/svg/RenderSVGContainer.cpp

Issue 360833002: Divorce PaintInvalidationState from LayoutState (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Address Julien's comments Created 6 years, 5 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org> 2 * Copyright (C) 2004, 2005, 2007 Nikolas Zimmermann <zimmermann@kde.org>
3 * Copyright (C) 2004, 2005, 2007, 2008 Rob Buis <buis@kde.org> 3 * Copyright (C) 2004, 2005, 2007, 2008 Rob Buis <buis@kde.org>
4 * Copyright (C) 2007 Eric Seidel <eric@webkit.org> 4 * Copyright (C) 2007 Eric Seidel <eric@webkit.org>
5 * Copyright (C) 2009 Google, Inc. All rights reserved. 5 * Copyright (C) 2009 Google, Inc. All rights reserved.
6 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org> 6 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
7 * 7 *
8 * This library is free software; you can redistribute it and/or 8 * This library is free software; you can redistribute it and/or
9 * modify it under the terms of the GNU Library General Public 9 * modify it under the terms of the GNU Library General Public
10 * License as published by the Free Software Foundation; either 10 * License as published by the Free Software Foundation; either
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
45 } 45 }
46 46
47 RenderSVGContainer::~RenderSVGContainer() 47 RenderSVGContainer::~RenderSVGContainer()
48 { 48 {
49 } 49 }
50 50
51 void RenderSVGContainer::layout() 51 void RenderSVGContainer::layout()
52 { 52 {
53 ASSERT(needsLayout()); 53 ASSERT(needsLayout());
54 54
55 // RenderSVGRoot disables layoutState for the SVG rendering tree.
56 ASSERT(!view()->layoutStateCachedOffsetsEnabled());
57
58 // Allow RenderSVGViewportContainer to update its viewport. 55 // Allow RenderSVGViewportContainer to update its viewport.
59 calcViewport(); 56 calcViewport();
60 57
61 // Allow RenderSVGTransformableContainer to update its transform. 58 // Allow RenderSVGTransformableContainer to update its transform.
62 bool updatedTransform = calculateLocalTransform(); 59 bool updatedTransform = calculateLocalTransform();
63 60
64 // RenderSVGViewportContainer needs to set the 'layout size changed' flag. 61 // RenderSVGViewportContainer needs to set the 'layout size changed' flag.
65 determineIfLayoutSizeChanged(); 62 determineIfLayoutSizeChanged();
66 63
67 SVGRenderSupport::layoutChildren(this, selfNeedsLayout() || SVGRenderSupport ::filtersForceContainerLayout(this)); 64 SVGRenderSupport::layoutChildren(this, selfNeedsLayout() || SVGRenderSupport ::filtersForceContainerLayout(this));
(...skipping 123 matching lines...) Expand 10 before | Expand all | Expand 10 after
191 if (objectBoundingBox().contains(localPoint)) { 188 if (objectBoundingBox().contains(localPoint)) {
192 updateHitTestResult(result, roundedLayoutPoint(localPoint)); 189 updateHitTestResult(result, roundedLayoutPoint(localPoint));
193 return true; 190 return true;
194 } 191 }
195 } 192 }
196 // 16.4: "If there are no graphics elements whose relevant graphics content is under the pointer (i.e., there is no target element), the event is not dispat ched." 193 // 16.4: "If there are no graphics elements whose relevant graphics content is under the pointer (i.e., there is no target element), the event is not dispat ched."
197 return false; 194 return false;
198 } 195 }
199 196
200 } 197 }
OLDNEW
« no previous file with comments | « Source/core/rendering/svg/RenderSVGBlock.cpp ('k') | Source/core/rendering/svg/RenderSVGForeignObject.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698