Refactoring out the code in LayoutTreeAsText::writeLayoutObject.

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

Index: third_party/WebKit/Source/core/layout/LayoutObject.cpp
diff --git a/third_party/WebKit/Source/core/layout/LayoutObject.cpp b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
index 5039f4bcc8dd294fffed8b9fe8257cdef27623d3..b8360a2d2c6ba826129affb43b3da08c829866dd 100644
--- a/third_party/WebKit/Source/core/layout/LayoutObject.cpp
+++ b/third_party/WebKit/Source/core/layout/LayoutObject.cpp
@@ -77,6 +77,7 @@
 #include "platform/RuntimeEnabledFeatures.h"
 #include "platform/TracedValue.h"
 #include "platform/geometry/TransformState.h"
+#include "platform/scroll/ScrollableArea.h"
 #include "wtf/allocator/Partitions.h"
 #include "wtf/text/StringBuilder.h"
 #include "wtf/text/WTFString.h"
@@ -3200,6 +3201,14 @@ ObjectPaintProperties& LayoutObject::ensureObjectPaintProperties()
     return *addResult.storedValue->value;
 }
 
+void LayoutObject::setLayoutRectDimForLayoutView(LayoutRect& rect) const
+{
+    if (isLayoutView()) {

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

Should this be moved to a virtual method so that LayoutView can override it?

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

Could you please clarify on this? So to create a virtual method that essentially
do nothing, but LayoutView override it to update the width and height? 

Can a LayoutObject be both a LayoutText/LayoutInline/LayoutTableCell/LayoutBox
and a LayoutView? 

+        rect.setWidth(LayoutUnit(toLayoutView(this)->viewWidth(IncludeScrollbars)));
+        rect.setHeight(LayoutUnit(toLayoutView(this)->viewHeight(IncludeScrollbars)));
+    }
+}
+
 } // namespace blink
 
 #ifndef NDEBUG