Refactoring out the code in LayoutTreeAsText::writeLayoutObject.

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

Index: third_party/WebKit/Source/core/layout/LayoutInline.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutInline.cpp b/third_party/WebKit/Source/core/layout/LayoutInline.cpp
index 5e8c2189902f9de46259c5ca3ab902728e1d423c..cfd4e6867a7c3321f5afacbcaa0a42888fe0f723 100644
--- a/third_party/WebKit/Source/core/layout/LayoutInline.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutInline.cpp
@@ -1369,4 +1369,13 @@ void LayoutInline::invalidateDisplayItemClients(PaintInvalidationReason invalida
         paintInvalidator.invalidateDisplayItemClient(*box, invalidationReason);
 }
 
+// FIXME: Would be better not to just dump 0, 0 as the x and y here.

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

It looks like this FIXME has been addressed?

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

Done.

+LayoutRect LayoutInline::debugRect() const
+{
+    IntRect linesBox = enclosingIntRect(linesBoundingBox());

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

Why aren't we just returning linesBoundingBox()?  This will lose subpixel
precision.

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

Done.

+    LayoutRect r = LayoutRect(IntRect(IntRect(linesBox.x(), linesBox.y(), linesBox.width(), linesBox.height())));

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

Remove redundant cast to IntRect.

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

Done.

+
+    return r;
+}
+
 } // namespace blink