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

Unified Diff: Source/core/layout/LayoutObject.cpp

Issue 1224933002: Combine outline and focus ring code paths (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix empty focus ring on Mac Created 5 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/layout/LayoutInline.cpp ('k') | Source/core/layout/svg/LayoutSVGModelObject.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/layout/LayoutObject.cpp
diff --git a/Source/core/layout/LayoutObject.cpp b/Source/core/layout/LayoutObject.cpp
index b52831dc07c8ffca5d8be540ac8bbf672d5559d8..b197aefe32cca310ed1bc3fc7369cfddb28585bd 100644
--- a/Source/core/layout/LayoutObject.cpp
+++ b/Source/core/layout/LayoutObject.cpp
@@ -1406,11 +1406,10 @@ PaintInvalidationReason LayoutObject::paintInvalidationReason(const LayoutBoxMod
if (shouldDoFullPaintInvalidation())
return m_bitfields.fullPaintInvalidationReason();
- // The focus ring may change because of position change of descendants. For simplicity,
+ // The outline may change shape because of position change of descendants. For simplicity,
// just force full paint invalidation if this object is marked for checking paint invalidation
// for any reason.
- // TODO(wangxianzhu): extend this to all outlines.
- if (styleRef().outlineStyleIsAuto())
+ if (styleRef().hasOutline())
return PaintInvalidationOutline;
// If the bounds are the same then we know that none of the statements below
« no previous file with comments | « Source/core/layout/LayoutInline.cpp ('k') | Source/core/layout/svg/LayoutSVGModelObject.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698