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

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

Issue 335963002: Change LayoutState to be stack-allocated (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix assert Created 6 years, 6 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 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 47
48 RenderSVGContainer::~RenderSVGContainer() 48 RenderSVGContainer::~RenderSVGContainer()
49 { 49 {
50 } 50 }
51 51
52 void RenderSVGContainer::layout() 52 void RenderSVGContainer::layout()
53 { 53 {
54 ASSERT(needsLayout()); 54 ASSERT(needsLayout());
55 55
56 // RenderSVGRoot disables layoutState for the SVG rendering tree. 56 // RenderSVGRoot disables layoutState for the SVG rendering tree.
57 ASSERT(!view()->layoutStateEnabled()); 57 ASSERT(!view()->layoutStateCachedOffsetsEnabled());
58 58
59 LayoutRepainter repainter(*this, SVGRenderSupport::checkForSVGRepaintDuringL ayout(this) || selfWillPaint()); 59 LayoutRepainter repainter(*this, SVGRenderSupport::checkForSVGRepaintDuringL ayout(this) || selfWillPaint());
60 60
61 // Allow RenderSVGViewportContainer to update its viewport. 61 // Allow RenderSVGViewportContainer to update its viewport.
62 calcViewport(); 62 calcViewport();
63 63
64 // Allow RenderSVGTransformableContainer to update its transform. 64 // Allow RenderSVGTransformableContainer to update its transform.
65 bool updatedTransform = calculateLocalTransform(); 65 bool updatedTransform = calculateLocalTransform();
66 66
67 // RenderSVGViewportContainer needs to set the 'layout size changed' flag. 67 // RenderSVGViewportContainer needs to set the 'layout size changed' flag.
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
197 if (objectBoundingBox().contains(localPoint)) { 197 if (objectBoundingBox().contains(localPoint)) {
198 updateHitTestResult(result, roundedLayoutPoint(localPoint)); 198 updateHitTestResult(result, roundedLayoutPoint(localPoint));
199 return true; 199 return true;
200 } 200 }
201 } 201 }
202 // 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 // 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 return false; 203 return false;
204 } 204 }
205 205
206 } 206 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698