Don't skip table rows when computing visual rects if the row is the ancestor.

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

Index: third_party/WebKit/Source/core/layout/LayoutBox.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutBox.cpp b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
index bd3dd55b2e8ccb76b136b641254925cee03264ec..ea7a6736c2960f442763bbdc403f18836e123dba 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
@@ -2045,7 +2045,7 @@ bool LayoutBox::mapToVisualRectInAncestorSpace(const LayoutBoxModelObject* ances
     LayoutObject* container = this->container(ancestor, &ancestorSkipped, &filterOrReflectionSkipped);
     LayoutBox* localContainingBlock = containingBlock();
     // Skip table row because cells and rows are in the same coordinate space.
-    if (container->isTableRow()) {
+    if (container->isTableRow() && container != ancestor) {

[wkorman, 2016/07/15 21:04:30]

Perhaps add or update the comment to note that we don't skip if the row is both
container and ancestor so as to handle the composited case.

[chrishtr, 2016/07/15 21:09:08]

Done.

         DCHECK(isTableCell());
         localContainingBlock = toLayoutBox(container->parent());
         container = container->parent();