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

Side by Side Diff: Source/core/layout/svg/LayoutSVGShape.cpp

Issue 1142283004: Implement a Hit Test Cache. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 5 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
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, 2008 Rob Buis <buis@kde.org> 3 * Copyright (C) 2004, 2005, 2008 Rob Buis <buis@kde.org>
4 * Copyright (C) 2005, 2007 Eric Seidel <eric@webkit.org> 4 * Copyright (C) 2005, 2007 Eric Seidel <eric@webkit.org>
5 * Copyright (C) 2009 Google, Inc. 5 * Copyright (C) 2009 Google, Inc.
6 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org> 6 * Copyright (C) 2009 Dirk Schulze <krit@webkit.org>
7 * Copyright (C) Research In Motion Limited 2010. All rights reserved. 7 * Copyright (C) Research In Motion Limited 2010. All rights reserved.
8 * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com> 8 * Copyright (C) 2009 Jeff Schiller <codedread@gmail.com>
9 * Copyright (C) 2011 Renata Hodovan <reni@webkit.org> 9 * Copyright (C) 2011 Renata Hodovan <reni@webkit.org>
10 * Copyright (C) 2011 University of Szeged 10 * Copyright (C) 2011 University of Szeged
(...skipping 203 matching lines...) Expand 10 before | Expand all | Expand 10 after
214 // We only draw in the foreground phase, so we only hit-test then. 214 // We only draw in the foreground phase, so we only hit-test then.
215 if (hitTestAction != HitTestForeground) 215 if (hitTestAction != HitTestForeground)
216 return false; 216 return false;
217 217
218 FloatPoint localPoint; 218 FloatPoint localPoint;
219 if (!SVGLayoutSupport::transformToUserSpaceAndCheckClipping(this, localToPar entTransform(), pointInParent, localPoint)) 219 if (!SVGLayoutSupport::transformToUserSpaceAndCheckClipping(this, localToPar entTransform(), pointInParent, localPoint))
220 return false; 220 return false;
221 221
222 PointerEventsHitRules hitRules(PointerEventsHitRules::SVG_GEOMETRY_HITTESTIN G, result.hitTestRequest(), style()->pointerEvents()); 222 PointerEventsHitRules hitRules(PointerEventsHitRules::SVG_GEOMETRY_HITTESTIN G, result.hitTestRequest(), style()->pointerEvents());
223 if (nodeAtFloatPointInternal(result.hitTestRequest(), localPoint, hitRules)) { 223 if (nodeAtFloatPointInternal(result.hitTestRequest(), localPoint, hitRules)) {
224 updateHitTestResult(result, roundedLayoutPoint(localPoint)); 224 updateHitTestResult(result, roundedLayoutPoint(localPoint), boundingRect (result.hitTestLocation().point()));
225 return true; 225 return true;
226 } 226 }
227 227
228 return false; 228 return false;
229 } 229 }
230 230
231 bool LayoutSVGShape::nodeAtFloatPointInternal(const HitTestRequest& request, con st FloatPoint& localPoint, PointerEventsHitRules hitRules) 231 bool LayoutSVGShape::nodeAtFloatPointInternal(const HitTestRequest& request, con st FloatPoint& localPoint, PointerEventsHitRules hitRules)
232 { 232 {
233 bool isVisible = (style()->visibility() == VISIBLE); 233 bool isVisible = (style()->visibility() == VISIBLE);
234 if (isVisible || !hitRules.requireVisible) { 234 if (isVisible || !hitRules.requireVisible) {
(...skipping 46 matching lines...) Expand 10 before | Expand all | Expand 10 after
281 SVGLayoutSupport::intersectPaintInvalidationRectWithResources(this, m_paintI nvalidationBoundingBox); 281 SVGLayoutSupport::intersectPaintInvalidationRectWithResources(this, m_paintI nvalidationBoundingBox);
282 } 282 }
283 283
284 float LayoutSVGShape::strokeWidth() const 284 float LayoutSVGShape::strokeWidth() const
285 { 285 {
286 SVGLengthContext lengthContext(element()); 286 SVGLengthContext lengthContext(element());
287 return lengthContext.valueForLength(style()->svgStyle().strokeWidth()); 287 return lengthContext.valueForLength(style()->svgStyle().strokeWidth());
288 } 288 }
289 289
290 } 290 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698