Refactoring out the code in LayoutTreeAsText::writeLayoutObject.

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 c2163a8009c8d303946bc1a329cfbd8110711df5..4b8982265809017f9cbc51c9aa04a73387368145 100644
--- a/third_party/WebKit/Source/core/layout/LayoutBox.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutBox.cpp
@@ -4866,4 +4866,18 @@ SnapAreaSet* LayoutBox::snapAreas() const
     return m_rareData ? m_rareData->m_snapAreas.get() : nullptr;
 }
 
+LayoutRect LayoutBox::debugRect() const
+{
+    LayoutRect r = frameRect();
+
+    // FIXME: Temporary in order to ensure compatibility with existing layout
+    // test results.
+    LayoutBlock* cb = containingBlock();
+    if (cb && cb->isTableCell())

[benjhayden, 2016/08/29 20:45:27]

Should this be moved to a virtual method so that you don't need to check
isTableCell and then call toLayoutTableCell? 

[lunalu1, 2016/08/30 20:35:30]

I moved this part to a virtual method but I don't think I did it right. Could
you please be more clear about how to make this part virtual?

Thanks 

+        r.move(0, -toLayoutTableCell(containingBlock())->intrinsicPaddingBefore());
+
+    setLayoutRectDimForLayoutView(r);
+    return r;
+}
+
 } // namespace blink