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

Side by Side Diff: third_party/WebKit/Source/core/layout/svg/LayoutSVGRoot.cpp

Issue 2727843004: needsPaintInvalidation() should not return true for selection. (Closed)
Patch Set: Renaming. Created 3 years, 9 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, 2007, 2008, 2009 Rob Buis <buis@kde.org> 3 * Copyright (C) 2004, 2005, 2007, 2008, 2009 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. 5 * Copyright (C) 2009 Google, Inc.
6 * Copyright (C) Research In Motion Limited 2011. All rights reserved. 6 * Copyright (C) Research In Motion Limited 2011. All rights reserved.
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 230 matching lines...) Expand 10 before | Expand all | Expand 10 after
241 241
242 void LayoutSVGRoot::willBeDestroyed() { 242 void LayoutSVGRoot::willBeDestroyed() {
243 SVGResourcesCache::clientDestroyed(this); 243 SVGResourcesCache::clientDestroyed(this);
244 LayoutReplaced::willBeDestroyed(); 244 LayoutReplaced::willBeDestroyed();
245 } 245 }
246 246
247 void LayoutSVGRoot::styleDidChange(StyleDifference diff, 247 void LayoutSVGRoot::styleDidChange(StyleDifference diff,
248 const ComputedStyle* oldStyle) { 248 const ComputedStyle* oldStyle) {
249 if (diff.needsFullLayout()) 249 if (diff.needsFullLayout())
250 setNeedsBoundariesUpdate(); 250 setNeedsBoundariesUpdate();
251 if (diff.needsPaintInvalidation()) { 251 if (diff.needsFullPaintInvalidation()) {
252 // Box decorations may have appeared/disappeared - recompute status. 252 // Box decorations may have appeared/disappeared - recompute status.
253 m_hasBoxDecorationBackground = styleRef().hasBoxDecorationBackground(); 253 m_hasBoxDecorationBackground = styleRef().hasBoxDecorationBackground();
254 } 254 }
255 255
256 LayoutReplaced::styleDidChange(diff, oldStyle); 256 LayoutReplaced::styleDidChange(diff, oldStyle);
257 SVGResourcesCache::clientStyleChanged(this, diff, styleRef()); 257 SVGResourcesCache::clientStyleChanged(this, diff, styleRef());
258 } 258 }
259 259
260 bool LayoutSVGRoot::isChildAllowed(LayoutObject* child, 260 bool LayoutSVGRoot::isChildAllowed(LayoutObject* child,
261 const ComputedStyle&) const { 261 const ComputedStyle&) const {
(...skipping 222 matching lines...) Expand 10 before | Expand all | Expand 10 after
484 if (result.addNodeToListBasedTestResult(node(), locationInContainer, 484 if (result.addNodeToListBasedTestResult(node(), locationInContainer,
485 boundsRect) == StopHitTesting) 485 boundsRect) == StopHitTesting)
486 return true; 486 return true;
487 } 487 }
488 } 488 }
489 489
490 return false; 490 return false;
491 } 491 }
492 492
493 } // namespace blink 493 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/layout/LayoutScrollbarPart.cpp ('k') | third_party/WebKit/Source/core/style/ComputedStyle.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698