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

Unified Diff: third_party/WebKit/Source/core/paint/BoxPainter.cpp

Issue 1983853003: Refactor background painting behind table cells (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/paint/BoxPainter.cpp
diff --git a/third_party/WebKit/Source/core/paint/BoxPainter.cpp b/third_party/WebKit/Source/core/paint/BoxPainter.cpp
index 9742626bb99f5824eb6d33ac320f87231f88dc2c..ea5b1376faa194979809ba28401a7fdea1c10c0e 100644
--- a/third_party/WebKit/Source/core/paint/BoxPainter.cpp
+++ b/third_party/WebKit/Source/core/paint/BoxPainter.cpp
@@ -51,11 +51,10 @@ void BoxPainter::paintBoxDecorationBackground(const PaintInfo& paintInfo, const
paintBoxDecorationBackgroundWithRect(paintInfo, paintOffset, paintRect);
}
-LayoutRect BoxPainter::boundsForDrawingRecorder(const LayoutPoint& paintOffset)
+LayoutRect BoxPainter::boundsForDrawingRecorder(const LayoutPoint& adjustedPaintOffset)
{
- // Use the visual overflow rect here, because it will include overflow introduced by the theme.
Xianzhu 2016/05/16 21:47:53 The comment doesn't look to make sense to me. We d
LayoutRect bounds = m_layoutBox.visualOverflowRect();
- bounds.moveBy(paintOffset);
+ bounds.moveBy(adjustedPaintOffset);
return bounds;
}

Powered by Google App Engine
This is Rietveld 408576698