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

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

Issue 264963004: Mark when we may have been invalidated to early out on repaintTreeAfterLayout. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 7 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 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
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. 55 // RenderSVGRoot disables layoutState for the SVG rendering tree.
56 ASSERT(!view()->layoutStateEnabled()); 56 ASSERT(!view()->layoutStateEnabled());
57 57
58 LayoutRepainter repainter(*this, SVGRenderSupport::checkForSVGRepaintDuringL ayout(this) || selfWillPaint()); 58 LayoutRepainter repainter(*this, SVGRenderSupport::checkForSVGRepaintDuringL ayout(this) || selfWillPaint());
59 59
60 setMayNeedInvalidation(true);
61
60 // Allow RenderSVGViewportContainer to update its viewport. 62 // Allow RenderSVGViewportContainer to update its viewport.
61 calcViewport(); 63 calcViewport();
62 64
63 // Allow RenderSVGTransformableContainer to update its transform. 65 // Allow RenderSVGTransformableContainer to update its transform.
64 bool updatedTransform = calculateLocalTransform(); 66 bool updatedTransform = calculateLocalTransform();
65 67
66 // RenderSVGViewportContainer needs to set the 'layout size changed' flag. 68 // RenderSVGViewportContainer needs to set the 'layout size changed' flag.
67 determineIfLayoutSizeChanged(); 69 determineIfLayoutSizeChanged();
68 70
69 SVGRenderSupport::layoutChildren(this, selfNeedsLayout() || SVGRenderSupport ::filtersForceContainerLayout(this)); 71 SVGRenderSupport::layoutChildren(this, selfNeedsLayout() || SVGRenderSupport ::filtersForceContainerLayout(this));
(...skipping 126 matching lines...) Expand 10 before | Expand all | Expand 10 after
196 if (objectBoundingBox().contains(localPoint)) { 198 if (objectBoundingBox().contains(localPoint)) {
197 updateHitTestResult(result, roundedLayoutPoint(localPoint)); 199 updateHitTestResult(result, roundedLayoutPoint(localPoint));
198 return true; 200 return true;
199 } 201 }
200 } 202 }
201 // 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." 203 // 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."
202 return false; 204 return false;
203 } 205 }
204 206
205 } 207 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698