Refactoring out the code in LayoutTreeAsText::writeLayoutObject.

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

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2005 Allan Sandfeld Jensen (kde@carewolf.com)
  *           (C) 2005, 2006 Samuel Weinig (sam.weinig@gmail.com)
  * Copyright (C) 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Adobe Systems Incorporated. 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 4833 matching lines @@
     if (m_rareData && m_rareData->m_snapAreas) {
         m_rareData->m_snapAreas->remove(&snapArea);
     }
 }
 
 SnapAreaSet* LayoutBox::snapAreas() const
 {
     return m_rareData ? m_rareData->m_snapAreas.get() : nullptr;
 }
 
+LayoutRect LayoutBox::debugRect() const
+{
+    LayoutRect r = frameRect();

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

We generally avoid single-letter abbreviations in variable names.  Can you call
this "rect" (and "block" for the containing block below)?

Same in the other files that do this.

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

Done.

+
+    LayoutBlock* cb = containingBlock();
+    if (cb)
+        cb->adjustChildDebugRect(r);
+
+    return r;
+}
+
 } // namespace blink