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

Unified Diff: Source/core/rendering/RenderTableCell.cpp

Issue 576823004: Move painting code for tables into *Paint classes. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Fix. Created 6 years, 3 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: Source/core/rendering/RenderTableCell.cpp
diff --git a/Source/core/rendering/RenderTableCell.cpp b/Source/core/rendering/RenderTableCell.cpp
index a0c76d2f46266dbb8d17f61edb21c068eb2f3028..86af82d2fe550b4fc4e9728e6407a593ad92ba8f 100644
--- a/Source/core/rendering/RenderTableCell.cpp
+++ b/Source/core/rendering/RenderTableCell.cpp
@@ -992,7 +992,7 @@ int RenderTableCell::borderHalfAfter(bool outer) const
void RenderTableCell::paint(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
{
ASSERT(paintInfo.phase != PaintPhaseCollapsedTableBorders);
- RenderBlockFlow::paint(paintInfo, paintOffset);
+ RenderBlock::paint(paintInfo, paintOffset);
mstensho (USE GERRIT) 2014/09/19 10:50:11 Why this change? While there's currently no implem
chrishtr 2014/09/19 16:35:24 I want it to be explicit, so that I can more easil
mstensho (USE GERRIT) 2014/09/19 18:08:55 But a table cell IS a RenderBlockFlow, and it may
chrishtr 2014/09/19 18:14:36 I don't care that much, so reverted this change.
}
static EBorderStyle collapsedBorderStyle(EBorderStyle style)

Powered by Google App Engine
This is Rietveld 408576698