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

Unified Diff: Source/core/paint/ObjectPainter.cpp

Issue 995583002: [S.P.] Use a correct bounding box for outlines of LayoutInlines for cull rects. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/paint/ObjectPainter.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/paint/ObjectPainter.cpp
diff --git a/Source/core/paint/ObjectPainter.cpp b/Source/core/paint/ObjectPainter.cpp
index a5342386e786d5f25b4927b55285da19fc3ee938..4a5fd602d35494b5ff530a8e48750a58a4e69b00 100644
--- a/Source/core/paint/ObjectPainter.cpp
+++ b/Source/core/paint/ObjectPainter.cpp
@@ -15,10 +15,8 @@
namespace blink {
-void ObjectPainter::paintFocusRing(const PaintInfo& paintInfo, const LayoutPoint& paintOffset, const LayoutStyle& style)
+void ObjectPainter::paintFocusRing(const PaintInfo& paintInfo, const LayoutStyle& style, const Vector<LayoutRect>& focusRingRects)
{
- Vector<LayoutRect> focusRingRects;
- m_layoutObject.addFocusRingRects(focusRingRects, paintOffset);
ASSERT(style.outlineStyleIsAuto());
Vector<IntRect> focusRingIntRects;
for (size_t i = 0; i < focusRingRects.size(); ++i)
@@ -39,7 +37,9 @@ void ObjectPainter::paintOutline(const PaintInfo& paintInfo, const LayoutRect& o
if (styleToUse.outlineStyleIsAuto()) {
if (LayoutTheme::theme().shouldDrawDefaultFocusRing(&m_layoutObject)) {
// Only paint the focus ring by hand if the theme isn't able to draw the focus ring.
- paintFocusRing(paintInfo, objectBounds.location(), styleToUse);
+ Vector<LayoutRect> focusRingRects;
+ m_layoutObject.addFocusRingRects(focusRingRects, objectBounds.location());
+ paintFocusRing(paintInfo, styleToUse, focusRingRects);
}
return;
}
« no previous file with comments | « Source/core/paint/ObjectPainter.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698