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

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

Issue 2392443009: reflow comments in core/paint (Closed)
Patch Set: Created 4 years, 2 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/TablePaintInvalidator.cpp
diff --git a/third_party/WebKit/Source/core/paint/TablePaintInvalidator.cpp b/third_party/WebKit/Source/core/paint/TablePaintInvalidator.cpp
index ab715940f231d0ad4514d338c04688fce834aeee..ae42aec7653bb485b37f094488470f9829c2f779 100644
--- a/third_party/WebKit/Source/core/paint/TablePaintInvalidator.cpp
+++ b/third_party/WebKit/Source/core/paint/TablePaintInvalidator.cpp
@@ -18,11 +18,13 @@ PaintInvalidationReason TablePaintInvalidator::invalidatePaintIfNeeded() {
PaintInvalidationReason reason =
BoxPaintInvalidator(m_table, m_context).invalidatePaintIfNeeded();
- // Table cells paint background from the containing column group, column, section and row.
- // If background of any of them changed, we need to invalidate all affected cells.
- // Here use shouldDoFullPaintInvalidation() as a broader condition of background change.
+ // Table cells paint background from the containing column group, column,
+ // section and row. If background of any of them changed, we need to
+ // invalidate all affected cells. Here use shouldDoFullPaintInvalidation() as
+ // a broader condition of background change.
- // If any col changed background, we'll check all cells for background changes.
+ // If any col changed background, we'll check all cells for background
+ // changes.
bool hasColChangedBackground = false;
for (LayoutTableCol* col = m_table.firstColumn(); col;
col = col->nextColumn()) {
@@ -49,8 +51,9 @@ PaintInvalidationReason TablePaintInvalidator::invalidatePaintIfNeeded() {
for (LayoutTableCell* cell = row->firstCell(); cell;
cell = cell->nextCell()) {
bool invalidated = false;
- // Table cells paint container's background on the container's backing instead of its own (if any),
- // so we must invalidate it by the containers.
+ // Table cells paint container's background on the container's backing
+ // instead of its own (if any), so we must invalidate it by the
+ // containers.
if (section->backgroundChangedSinceLastPaintInvalidation()) {
sectionInvalidator
.slowSetPaintingLayerNeedsRepaintAndInvalidateDisplayItemClient(
« no previous file with comments | « third_party/WebKit/Source/core/paint/TableCellPainter.cpp ('k') | third_party/WebKit/Source/core/paint/TablePainter.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698