Changeset 205920 in webkit

Timestamp:

Sep 14, 2016 11:46:32 AM (8 years ago)

Author:

Alan Bujtas

Message:

ShowRenderTree should take position offset into account when printing inflow positioned renderers.
​https://bugs.webkit.org/show_bug.cgi?id=161978

Reviewed by Simon Fraser.

Adjust (x, y) with the inflow positioned renderer's offset.

Not testable.

• rendering/RenderObject.cpp:

(WebCore::RenderObject::showRenderObject):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• rendering/RenderObject.cpp (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2016-09-14  Zalan Bujtas  <zalan@apple.com>
+
+        ShowRenderTree should take position offset into account when printing inflow positioned renderers.
+        https://bugs.webkit.org/show_bug.cgi?id=161978
+
+        Reviewed by Simon Fraser.
+
+        Adjust (x, y) with the inflow positioned renderer's offset.  
+
+        Not testable.
+
+        * rendering/RenderObject.cpp:
+        (WebCore::RenderObject::showRenderObject):
+
 2016-09-14  Alex Christensen  <achristensen@webkit.org>
 

trunk/Source/WebCore/rendering/RenderObject.cpp

@@ -1117 +1117 @@
 
     if (is<RenderBox>(*this)) {
-        FloatRect boxRect = downcast<RenderBox>(*this).frameRect();
+        auto& renderBox = downcast<RenderBox>(*this);
+        FloatRect boxRect = renderBox.frameRect();
+        if (renderBox.isInFlowPositioned())
+            boxRect.move(renderBox.offsetForInFlowPosition());
         fprintf(stderr, "  (%.2f, %.2f) (%.2f, %.2f)", boxRect.x(), boxRect.y(), boxRect.width(), boxRect.height());
     } else if (is<RenderInline>(*this) && isInFlowPositioned()) {