Refactoring out the code in LayoutTreeAsText::writeLayoutObject.

third_party/WebKit/Source/core/layout/LayoutTableCell.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, 2007, 2008, 2009 Apple Inc. All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
@@ skipping 1023 matching lines @@
 
 bool LayoutTableCell::backgroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect) const
 {
     // If this object has layer, the area of collapsed borders should be transparent
     // to expose the collapsed borders painted on the underlying layer.
     if (hasLayer() && table()->collapseBorders())
         return false;
     return LayoutBlockFlow::backgroundIsKnownToBeOpaqueInRect(localRect);
 }
 
+// FIXME: Deliberately dump the "inner" box of table cells, since that is what
+// current results reflect.  We'd like to clean up the results to dump both the
+// outer box and the intrinsic padding so that both bits of information are
+// captured by the results.
+LayoutRect LayoutTableCell::debugRect() const
+{
+    LayoutRect r = LayoutRect(location().x(), location().y() + intrinsicPaddingBefore(), size().width(), size().height() - intrinsicPaddingBefore() - intrinsicPaddingAfter());

[skobes, 2016/09/09 20:27:41]

Wrap this line at the commas for readability.

[lunalu1, 2016/09/12 14:45:14]

Done.

+
+    LayoutBlock* cb = containingBlock();
+    if (cb)
+        cb->adjustChildDebugRect(r);
+
+    return r;
+}
+
+void LayoutTableCell::adjustChildDebugRect(LayoutRect& r) const
+{
+    r.move(0, -intrinsicPaddingBefore());
+}
+
 } // namespace blink