Fix table cell background painting

third_party/WebKit/Source/core/layout/LayoutTableSection.cpp

 /*
  * Copyright (C) 1997 Martin Jones (mjones@kde.org)
  *           (C) 1997 Torben Weis (weis@kde.org)
  *           (C) 1998 Waldo Bastian (bastian@kde.org)
  *           (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  * Copyright (C) 2003, 2004, 2005, 2006, 2008, 2009, 2010, 2013 Apple Inc. All rights reserved.
  * Copyright (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 1629 matching lines @@
 
     // FIXME: The table's direction should determine our row's direction, not the section's (see bug 96691).
     if (!style()->isLeftToRightDirection())
         cellLocation.setX(LayoutUnit(table()->effectiveColumnPositions()[table()->numEffectiveColumns()] - table()->effectiveColumnPositions()[table()->absoluteColumnToEffectiveColumn(cell->absoluteColumnIndex() + cell->colSpan())] + horizontalBorderSpacing));
     else
         cellLocation.setX(LayoutUnit(table()->effectiveColumnPositions()[effectiveColumn] + horizontalBorderSpacing));
 
     cell->setLogicalLocation(cellLocation);
 }
 
+LayoutRect LayoutTableSection::getCellPosition(unsigned row, unsigned effectiveColumn) const
+{
+    LayoutTable * table = this->table();
+
+    LayoutUnit left = LayoutUnit(table->effectiveColumnPositions()[effectiveColumn]);
+    LayoutUnit top = LayoutUnit(m_rowPos[row]);
+    LayoutUnit right = LayoutUnit(table->effectiveColumnPositions()[effectiveColumn+1]);
+    LayoutUnit bottom = LayoutUnit(m_rowPos[row+1]);
+    left += LayoutUnit(table->hBorderSpacing());
+
+    return LayoutRect(left, top, right - left, bottom - top);
+}
+
+LayoutRect LayoutTableSection::getCellPhysicalPosition(unsigned row, unsigned effectiveColumn) const
+{
+    return transformLogicalToPhysicalPosition(getCellPosition(row, effectiveColumn));
+}
+
+LayoutRect LayoutTableSection::transformLogicalToPhysicalPosition(const LayoutRect& position) const

[Xianzhu, 2016/03/28 00:44:42]

Can you use LayoutBox::flipForWritingMode() in this function?

[atotic1, 2016/03/29 17:05:16]

No. The transformation computed here is more complex. This is the simplest
I could make it.

+{
+    WritingMode writingMode = style()->getWritingMode();
+    bool isLTR = style()->isLeftToRightDirection();
+
+    LayoutRect transformedPosition = position;
+    if (blink::isHorizontalWritingMode(writingMode)) {
+        if (!isLTR)
+            transformedPosition.setX(size().width() - transformedPosition.maxX());
+    } else {
+        transformedPosition = transformedPosition.transposedRect();
+        if (!isLTR)
+            transformedPosition.setY(size().height() - transformedPosition.maxY());
+        if (blink::isFlippedBlocksWritingMode(writingMode))
+            transformedPosition.setX(size().width() - transformedPosition.maxX());
+    }
+    return transformedPosition;
+}
+
+LayoutRect LayoutTableSection::positionByCellSpan() const
+{
+    LayoutRect position;
+    LayoutTableRow* firstRow = this->firstRow();
+    LayoutTableRow* lastRow = this->lastRow();
+    if (firstRow && lastRow) {
+        position = firstRow->positionByCellSpan();
+        position.unite(lastRow->positionByCellSpan());
+    }
+    return position;
+}
+
 } // namespace blink